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

adding contributing guideline [skip ci] #259

Merged
merged 2 commits into from Sep 22, 2023
Merged

adding contributing guideline [skip ci] #259

merged 2 commits into from Sep 22, 2023

Conversation

baentsch
Copy link
Member

Adding final item missing to complete https://github.com/open-quantum-safe/oqs-provider/community.

Feedback/suggestions for improvements particularly solicited from the most recent new contributors @thb-sb @marcbrevoort-cyberhive @WillChilds-Klein @alexzas: What information would help (have helped you :) most if documented?

@WillChilds-Klein
Copy link
Contributor

Hi @baentsch, thanks for following up here. The only hiccup I had in PR #188 was diagnosing CI failures, as CircleCI doesn't show failed test output (at least not to non-maintainers such as myself). Perhaps a sentence or two explaining how to run the tests locally would help newcomers diagnose their own issues before publishing a draft PR. Similarly, many projects I've seen have a DEVELOPMENT.md file explaining common workflows in the build/dev/test cycle.

@baentsch
Copy link
Member Author

Thanks @WillChilds-Klein for the suggestions. Please check out the latest additions to the PR and let me know if/where you'd see obvious further room for improvement.

Copy link
Contributor

@WillChilds-Klein WillChilds-Klein left a comment

Choose a reason for hiding this comment

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

Excellent documentation, thank you @baentsch!

@baentsch baentsch merged commit f332584 into main Sep 22, 2023
1 check passed
@baentsch baentsch deleted the mb-contributing branch September 22, 2023 05:51
feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 13, 2024
* adding contributing guideline

* adding further dev support information

Signed-off-by: Felipe Ventura <felipe.ventura@entrust.com>
feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 16, 2024
* adding contributing guideline

* adding further dev support information

Signed-off-by: Felipe Ventura <felipe.ventura@entrust.com>
feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 16, 2024
* adding contributing guideline

* adding further dev support information
feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 17, 2024
* adding contributing guideline

* adding further dev support information

Signed-off-by: Felipe Ventura <felipe.ventura@entrust.com>
feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 17, 2024
* adding contributing guideline

* adding further dev support information

Signed-off-by: Felipe Ventura <felipe.ventura@entrust.com>
feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 17, 2024
* adding contributing guideline

* adding further dev support information

Signed-off-by: Felipe Ventura <felipe.ventura@entrust.com>
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.

None yet

3 participants