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

Developer guide #660

Open
atkinsonm opened this issue Aug 21, 2023 · 12 comments
Open

Developer guide #660

atkinsonm opened this issue Aug 21, 2023 · 12 comments

Comments

@atkinsonm
Copy link
Contributor

It would be nice if there was a contribution and developer guide for this project that could document:

  • Recommended paths for development i.e. how to best set up your developer workspace for FE/BE development
  • Guidelines for PRs, what should be included, what generally will/will not be accepted, maybe a pull request template

I struggled a bit with frontend development on Windows (e.g. Chrome dev tools not picking up my local workspace, exacerbated slightly when I tried to use WSL) enough to where a paragraph or two with some development/testing tips would have been helpful.

@frank-weinberg
Copy link
Contributor

I agree that this would be nice to have. But I don't see myself having any meaningful resources for creating it in the foreseeable future.

@davidmreed
Copy link

I'd be interested in setting up a VS Code dev container-based contribution flow. Happy to contribute that if it sounds valuable.

@frank-weinberg
Copy link
Contributor

As long as it doesn't interfere with my non-container VS Code setup (which it probably won't), I don't mind including it. I don't know if anybody would be using it, but it won't hurt to have it available.

@aidanmontare-edu
Copy link

@davidmreed I'm currently getting started developing for the scoreboard using dev containers, did you wind up trying a setup like that?

@aidanmontare-edu
Copy link

Started a "Contributing to CRG ScoreBoard" draft here: https://github.com/rollerderby/scoreboard/wiki/Contributing-to-CRG-ScoreBoard

This might be a bit more general than the original issue, but I figured it might be nice to have a general contributing guide as well as more detailed developer guidance.

@atkinsonm
Copy link
Contributor Author

Started a "Contributing to CRG ScoreBoard" draft here: https://github.com/rollerderby/scoreboard/wiki/Contributing-to-CRG-ScoreBoard

This might be a bit more general than the original issue, but I figured it might be nice to have a general contributing guide as well as more detailed developer guidance.

@aidanmontare-edu this is a good start. What do you think about putting that content in .github/CONTRIBUTING.md? GitHub provides a few niceties for using that pattern that might help with discovery versus the wiki (though a more detailed dev guide that references the contributing guidelines would also be welcome).

@aidanmontare-edu
Copy link

aidanmontare-edu commented Jun 17, 2024 via email

@aidanmontare-edu
Copy link

@atkinsonm began a pull request, will write some more content soon!

@aidanmontare-edu
Copy link

aidanmontare-edu commented Jun 20, 2024

And here's a page for a more detailed developer guide: https://github.com/rollerderby/scoreboard/wiki/Developer-Guide (again, will try to add some actual content soon)

@aidanmontare-edu
Copy link

@davidmreed put my beginning of a DevContainers setup in #715. I'm just editing on the command line right now, so I image you might want to add more tools to get a good IDE experience.

@frank-weinberg
Copy link
Contributor

Just as a quick note from my side: I read over your drafts and agree with the approach you're taking so far. I'll look over the details and add my own inputs when I have more time. (Not before the fix for the groundhog day bug is at least PRed.)

@aidanmontare-edu
Copy link

Thanks, and much appreciated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants