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

ci: Linter and formatter #14

Merged
merged 7 commits into from
Oct 3, 2023
Merged

ci: Linter and formatter #14

merged 7 commits into from
Oct 3, 2023

Conversation

PedroChaparro
Copy link
Contributor

@PedroChaparro PedroChaparro commented Oct 2, 2023

Includes ✨

  • Install black formatter
  • Install ruff linter
  • Add recommended vscode extensions
  • Format and lint current files
  • Update integration pipeline

Related issues :octocat:

Closes #13

@PedroChaparro PedroChaparro added the ci Continuous integration label Oct 2, 2023
@PedroChaparro PedroChaparro self-assigned this Oct 2, 2023
Copy link
Member

@MiguelMRojas MiguelMRojas left a comment

Choose a reason for hiding this comment

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

Thanks for doing this PR, everything turned out great :)

@PedroChaparro PedroChaparro merged commit 22a84d2 into dev Oct 3, 2023
@PedroChaparro PedroChaparro deleted the ci/#13_linter_formatter branch October 3, 2023 21:04
@PedroChaparro PedroChaparro mentioned this pull request Oct 8, 2023
MiguelMRojas added a commit that referenced this pull request Oct 8, 2023
* docs: OpenApi Specification (#9)

* chore: Update docker-compose (#12)

* chore: Add missing env variables and services to docker-compose

* chore: Use worker default env variables

* ci: Linter and formatter (#14)

* chore: Add recommended vscode extensions

* chore(deps): Install black formatter and ruff linter

* style: Lint and format files

* docs: Add linting and formatting commands to README

* ci: Add linter and style checks to integration pipeline

* feat: endpoint login (#15)

* feat: endpoint challenge (#33)

---------

Co-authored-by: Antonio Donis <[email protected]>
Co-authored-by: Pedro Andrés Chaparro Quintero <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants