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

Repository setup #7

Open
2 of 4 tasks
Karanjot786 opened this issue Jun 21, 2024 · 6 comments
Open
2 of 4 tasks

Repository setup #7

Karanjot786 opened this issue Jun 21, 2024 · 6 comments
Assignees

Comments

@Karanjot786
Copy link
Member

Karanjot786 commented Jun 21, 2024

Description

We need to set up the repository for the WRROC Converter project properly. This includes setting up packaging, dependency management, linting, static code/type checks, API documentation, and a CI pipeline to ensure a robust development environment.

Tasks

@Karanjot786 Karanjot786 self-assigned this Jun 21, 2024
@uniqueg
Copy link
Member

uniqueg commented Jun 21, 2024

Thanks, @Karanjot786 - this looks very good. However, note that this issue will constitute more than one PR - which means that you can't link it to any individual PR (because addressing only one of the points shouldn't close the entire issue).

Best is to break it down and create individual issues for each of these points (1. - 4.), then convert this issue to a Task List issue, linking to the individual issues.

@uniqueg
Copy link
Member

uniqueg commented Jun 21, 2024

Because what you just now did (manually checking boxes, especially if they haven't even been merged) and linking this issue to one individual commit that does not fully address this issue (but by linking would close this issue upon merging the linked PR) is not very good.

If you have individual issues, it makes sense to link a PR to the issue and have it automatically closed when the PR is merged. And in a Task List, also the checkboxes are automatically checked when the referenced issue is closed.

@uniqueg uniqueg reopened this Jun 21, 2024
@uniqueg
Copy link
Member

uniqueg commented Jun 21, 2024

Re-opened the issue...

@uniqueg uniqueg changed the title Repository Setup for WRROC Converter Project Repository setup Jun 21, 2024
@uniqueg
Copy link
Member

uniqueg commented Jun 21, 2024

Looks good. Note that I edited the issue description to put the [ ] in a list so that they would render properly and create a task list.

@uniqueg
Copy link
Member

uniqueg commented Jun 21, 2024

Also, no need to include check boxes in the individual issues (you currently have them in two issues), just put a normal list. Otherwise you need to remember to check the boxes, and chances are that you will forget that. If an issue is closed with a PR, it is assumed that all points are addressed, or otherwise differences are explained in the PR (and project documentation, if necessary).

@Karanjot786
Copy link
Member Author

Thank you for your feedback. I will update the individual issues to use a normal list instead of checkboxes to avoid the need to manually check them off. This will ensure that closing an issue with a PR assumes all points are addressed or differences are explained in the PR and project documentation if necessary.

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

No branches or pull requests

2 participants