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

Contributing guidelines #1074

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Contributing guidelines #1074

wants to merge 2 commits into from

Conversation

alecandido
Copy link
Member

@alecandido alecandido commented Oct 16, 2024

This PR is mostly saving a couple of comments:

However, the idea is not to replace existing guides and so on, but just to (discuss and) establish our own conventions, when there is a lack of "official" guidelines (e.g. PEP 8).

Maintaining docs is an expensive process, so replacing a custom guide with a reliable externally maintained one would be always welcome. In which case we could replace our content with external links (which would be still useful to collect somewhere).

But worse than writing some documents (and maintain them) is not having anything at all, and just having to repeat (or worse: discuss again) everything every time.

In any case, this is mainly a sketch, even to trigger comments about the need of it. Any comment is more than welcome.

@alecandido alecandido requested a review from stavros11 October 16, 2024 12:17
Copy link

codecov bot commented Oct 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.55%. Comparing base (d16f03f) to head (367f773).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1074   +/-   ##
=======================================
  Coverage   50.55%   50.55%           
=======================================
  Files          63       63           
  Lines        2886     2886           
=======================================
  Hits         1459     1459           
  Misses       1427     1427           
Flag Coverage Δ
unittests 50.55% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Mainly to store some guidelines about imports, in the attempt to refine them
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.

1 participant