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

update template PR #469

Merged
merged 2 commits into from
Jan 7, 2024
Merged

Conversation

Mralexandresys
Copy link
Contributor

Pre-flight Checklist

Please ensure you've completed all of the following.

Description of Change

change url in template request
Add row check list for update package version

@@ -7,8 +7,9 @@

Please ensure you've completed all of the following.

- [ ] I have read the [Contributing Guidelines](https://github.com/ferdium/ferdium-recipes/blob/HEAD/CONTRIBUTING.md) for this project.
- [ ] I agree to follow the [Code of Conduct](https://github.com/ferdium/ferdium-recipes/blob/HEAD/CODE_OF_CONDUCT.md) that this project adheres to.
- [ ] I have read the [Contributing Guidelines](https://github.com/ferdium/ferdium-app/blob/develop/CONTRIBUTING.md) for this project.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please undo the 1st two lines of changes. Though the issues get reported in the main (ferdium-app) repo, the contriuting guidelines and scripts to run are different between the 2 repos.

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 understand what you mean but the url on the 1st line does not work. I used the CONTRIBUTING.md doc to install repo recipes on my local.
Should we create a new CONTRIBUTING.md ?

Copy link
Member

Choose a reason for hiding this comment

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

@Mralexandresys is right, the link to https://github.com/ferdium/ferdium-recipes/blob/HEAD/CONTRIBUTING.md is broken.

Maybe I fired too quicky in merging #471, @vraravam let us know if something has to be updated.

- [ ] I agree to follow the [Code of Conduct](https://github.com/ferdium/ferdium-recipes/blob/HEAD/CODE_OF_CONDUCT.md) that this project adheres to.
- [ ] I have read the [Contributing Guidelines](https://github.com/ferdium/ferdium-app/blob/develop/CONTRIBUTING.md) for this project.
- [ ] I agree to follow the [Code of Conduct](https://github.com/ferdium/ferdium-app/blob/develop/CODE_OF_CONDUCT.md) that this project adheres to.
- [ ] I update the version package [Updating](https://github.com/ferdium/ferdium-recipes/blob/main/docs/updating.md#4-updating-the-version-number)
Copy link
Member

Choose a reason for hiding this comment

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

Ah this is cool, then more ppl will hopefully do the version update right away without having to get reminded.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes! I would've appreciated knowing about this prior to the commit hook yelling at me about it

@SpecialAro SpecialAro self-assigned this Jan 7, 2024
@SpecialAro SpecialAro merged commit e26e06c into ferdium:main Jan 7, 2024
2 checks passed
@SpecialAro
Copy link
Member

@allcontributors - please add @Mralexandresys for infra

Copy link
Contributor

@SpecialAro

I've put up a pull request to add @Mralexandresys! 🎉

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.

5 participants