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

chore: version bump pr template #377

Merged
merged 5 commits into from
Nov 9, 2023
Merged

Conversation

andreabadesso
Copy link
Contributor

@andreabadesso andreabadesso commented Nov 3, 2023

Acceptance Criteria

  • We should have a default PR template with links to two PR templates -- the version bump and the feature branch PR templates
  • The version bump PR template should have a checklist for when creating a version bump in the wallet-mobile

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@andreabadesso andreabadesso self-assigned this Nov 3, 2023
@andreabadesso andreabadesso added the enhancement New feature or request label Nov 3, 2023
@@ -0,0 +1,13 @@
### Acceptance Criteria
- Include here all things that this PR should solve
Copy link
Contributor

@luislhl luislhl Nov 6, 2023

Choose a reason for hiding this comment

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

question: Should we change this to something like: "Include links to all PRs that will be included in this version"?

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 think we should, done in 6c72884

- [ ] Make sure you updated the `versionName` with the appropriate version, including the release candidate number in `android/app/build.gradle`

### Security Checklist
- [ ] Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.
Copy link
Contributor

Choose a reason for hiding this comment

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

thought: Maybe this is unnecessary in this template. What do you think?

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 agree, done in 259bf8a

.github/PULL_REQUEST_TEMPLATE.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants