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

Updates to the Developer Guide (Testing & Extending sections) #432

Merged
merged 8 commits into from
Jul 2, 2024

Conversation

sarina
Copy link
Contributor

@sarina sarina commented Nov 15, 2023

Updates some small typos, adds a section on GitHub Actions, and removes some outdated references related to testing and XBlocks

@sarina sarina changed the title Updates to the Developer Guide Updates to the Developer Guide (Testing & Extending sections) Nov 15, 2023
@sarina
Copy link
Contributor Author

sarina commented Nov 20, 2023

@feanil is there someone you can think of to review this that would know about testing & github actions?

@feanil
Copy link
Contributor

feanil commented Nov 27, 2023

Someone on arch-bom/arbi-bom at 2U might be a good candidate for that. @jmbowman do you have someone that you could recommend?

Copy link

@jmbowman jmbowman 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 to me other than maybe removing the mention of acceptance tests.

@sarina sarina force-pushed the sarina/more-dev-guide branch 4 times, most recently from 2d3bf67 to e0d85c9 Compare March 24, 2024 19:32
@sarina
Copy link
Contributor Author

sarina commented Mar 24, 2024

@feanil it's been 5 months but I have addressed Jeremy's one comment and got the build working again. \o/

I'll tag @brian-smith-tcril - Brian, you and I fixed up a good portion of the dev guide together. Any desire to take a run through this?

Copy link
Contributor

@brian-smith-tcril brian-smith-tcril left a comment

Choose a reason for hiding this comment

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

Left a few initial thoughts! Please let me know if you have any questions for me!

repository, or deploy merged pull requests to production.

The Open edX project uses Actions to automate test builds on pull requests, as
well as to create other types of automations. You can check out the workflows
Copy link
Contributor

Choose a reason for hiding this comment

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

as well as to create other types of automations

Maybe we could copy the first 2 paragraphs of the "Understanding GitHub Actions" page and make it clear we're directly quoting them, then expand this to be:

The Open edX project uses Actions to automate test builds on pull requests,
**manage/organize issues on project boards? something like that?**, and more!

available to all Open edX repositories in the `.github repo`_; many repos define
their own workflows in their ``.github/workflows`` directory.

Reading Builds
Copy link
Contributor

Choose a reason for hiding this comment

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

"Reading Builds" feels like odd phrasing to me. Maybe "Reading Build Status" or something?

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm having a hard time understanding the purpose of this file. It seems to be serving as an introduction to GitHub Actions as a concept, an overview of how GitHub Actions are used throughout the Open edX GH org, but also a couple how-tos for troubleshooting specific checks (GH Actions and otherwise) that exist on most repos.

I feel like we'd benefit from having:

  • What are GitHub Actions?
  • How are GitHub Actions used in the Open edX project?
  • How to troubleshoot? automated checks on repos in the Open edX project.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I restructured the document to have these three "sections" in this doc. I think it's a good framing.

@sarina sarina force-pushed the sarina/more-dev-guide branch from e0d85c9 to 59ea19c Compare July 2, 2024 11:58
@sarina sarina force-pushed the sarina/more-dev-guide branch from 59ea19c to a47e82f Compare July 2, 2024 12:22
@sarina sarina force-pushed the sarina/more-dev-guide branch from 0cafc92 to 1e701bd Compare July 2, 2024 12:47
@sarina sarina merged commit 6ef65a1 into main Jul 2, 2024
2 checks passed
@sarina sarina deleted the sarina/more-dev-guide branch July 2, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants