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

Updated contributing #1057

Merged
merged 2 commits into from
Sep 26, 2023
Merged

Updated contributing #1057

merged 2 commits into from
Sep 26, 2023

Conversation

marcus-oscarsson
Copy link
Member

  • Updated contributing guidelines so that to reflects the current state.
  • There is still a section missing for front end development for instance linting (but we also need to work on the linting tools)
  • There is, as well, a section missing about precomit and flake8 in addition to black


Before submiting the code to the repository please read these contributing guidelines.
The aim of these guidelines is to help the developers community to maintain the code stable and reusable.
Before submiting the code to the repository please read these contributing guidlines.
Copy link
Contributor

Choose a reason for hiding this comment

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

That should have remained 'guidelines', not 'guidlines'

CONTRIBUTING.md Outdated

* mm (millimeter) for translative motors and sizes
* degrees for rotative motors
* perecent (%) for ratios like attenuation
Copy link
Contributor

Choose a reason for hiding this comment

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

That should be 'percent'

Copy link
Contributor

@rhfogh rhfogh left a comment

Choose a reason for hiding this comment

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

Very good! Thanks.

@marcus-oscarsson marcus-oscarsson merged commit adfe227 into develop Sep 26, 2023
7 of 8 checks passed
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.

2 participants