Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

agda.section.md: Fix header and enumeration #98084

Merged
merged 2 commits into from Oct 11, 2020
Merged

Conversation

turion
Copy link
Contributor

@turion turion commented Sep 16, 2020

CC @alexarice

/marvin opt-in
/status needs_reviewer

@turion
Copy link
Contributor Author

turion commented Sep 16, 2020

The enumeration isn't fixed yet. Let me amend.

@turion
Copy link
Contributor Author

turion commented Sep 16, 2020

/marvin opt-in
/status needs_reviewer

@marvin-mk2 marvin-mk2 bot added the marvin label Sep 16, 2020
@marvin-mk2
Copy link

marvin-mk2 bot commented Sep 16, 2020

Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here.

Copy link
Contributor

@alexarice alexarice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching these, This probably needs to be backported once finished

doc/languages-frameworks/agda.section.md Outdated Show resolved Hide resolved
doc/languages-frameworks/agda.section.md Outdated Show resolved Hide resolved
Copy link
Contributor

@alexarice alexarice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, could you rebase down to 1 or 2 commits?

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-already-reviewed/2617/244

@kevincox kevincox merged commit 8b33d57 into NixOS:master Oct 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants