Page MenuHomeSolus

Initial commit of Alan
ClosedPublic

Authored by drakovyrn on Feb 12 2021, 6:34 AM.

Details

Summary

Initial commit of Alan. Resolves T9526

Test Plan

Build process of Alan automatically checks sample games for expected/unexpected output; currently there is only one regression

Diff Detail

Repository
R5163 alan
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

drakovyrn created this revision.Feb 12 2021, 6:34 AM
drakovyrn requested review of this revision.Feb 12 2021, 6:34 AM
drakovyrn updated this revision to Diff 25374.Feb 12 2021, 6:59 AM

Changed Clang key to 'yes'

JoshStrobl requested changes to this revision.Feb 20 2021, 10:37 PM
JoshStrobl added a subscriber: JoshStrobl.

LGTM aside from missing MAINTAINERS.md file which you were directed to in the task itself.

This revision now requires changes to proceed.Feb 20 2021, 10:37 PM

Okay, I'll revise that once I get home.

drakovyrn updated this revision to Diff 25517.Feb 23 2021, 9:51 PM

Add MAINTAINERS.md

algent added a subscriber: algent.Feb 24 2021, 7:39 PM

Hey @drakovyrn, you have to fix your MAINTAINERS.md file.

  1. Fix indentation, it should be 2 not 4.
  2. Remove matrix and leave only Name, IRC and email.

i think leaving matrix is fine?

@algent I'm leaving Matrix in there, Joshua told me I could put it there if I had one.

I'll fix the indentation real quick.

drakovyrn updated this revision to Diff 25531.Feb 24 2021, 8:07 PM

Fix MAINTAINERS.md indentation

JoshStrobl accepted this revision.Mar 6 2021, 3:51 PM
JoshStrobl edited the summary of this revision. (Show Details)

LGTM, thanks!

This revision is now accepted and ready to land.Mar 6 2021, 3:51 PM
This revision was automatically updated to reflect the committed changes.