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

DOC updated contributing guidelines #813

Merged
merged 2 commits into from
Jun 7, 2022
Merged

DOC updated contributing guidelines #813

merged 2 commits into from
Jun 7, 2022

Conversation

cyrusvahidi
Copy link
Collaborator

CONTRIBUTING.MD has been updated with guidelines on making feature requests and bug fixes

  • It explicitly outlines that an issue or feature request should be submitted before a PR
  • The community will discuss the feature request and its roadmap
  • Features for a PR should be planned to be small and manageable

@cyrusvahidi cyrusvahidi changed the title update contributing guidelines updated contributing guidelines May 20, 2022
@janden
Copy link
Collaborator

janden commented May 20, 2022

Can you also update the e-mail address in the code of conduct?

@MuawizChaudhary
Copy link
Collaborator

Can you also update the e-mail address in the code of conduct?

This pr should be committed to main perhaps.

soliciting community thoughts would also be a good idea on the code of conduct.

@cyrusvahidi
Copy link
Collaborator Author

done

@MuawizChaudhary MuawizChaudhary changed the title updated contributing guidelines DOC updated contributing guidelines May 31, 2022
Copy link
Collaborator

@janden janden left a comment

Choose a reason for hiding this comment

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

Thanks!

@janden janden merged commit 185f339 into kymatio:dev Jun 7, 2022
eickenberg pushed a commit that referenced this pull request Jul 5, 2022
* update contributing guidelines

* update email in code of conduct
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.

3 participants