Skip to content

Fix #2 - Setup CI/CD#8

Open
m-theus wants to merge 3 commits into
Computer-Progress:mainfrom
FGA-GCES:2-setup-ci/cd-for-backend
Open

Fix #2 - Setup CI/CD#8
m-theus wants to merge 3 commits into
Computer-Progress:mainfrom
FGA-GCES:2-setup-ci/cd-for-backend

Conversation

@m-theus

@m-theus m-theus commented Aug 12, 2021

Copy link
Copy Markdown

Description

This PR sets up three CI/CD workflows:

  • Build workflow
    Checks if the application build is running without errors.

  • Style workflow (with flake8)
    Checks if the application code follows the default style.

  • Test workflow (with coverage)
    Verifies that all application tests are passing and checks application testing coverage.

These workflows will run on every PR or push to main branch. And each workflow has a badge in the README to report its status.

Images

Badges

Captura de tela de 2021-08-12 02-58-09

Related Issue

#2

@m-theus m-theus changed the title 2 setup ci/cd for backend Fix #2 - Setup CI/CD Aug 12, 2021
@m-theus

m-theus commented Aug 22, 2021

Copy link
Copy Markdown
Author

Hi @mansogf,

Looking at the context (contribution to a community), I realized that I made some mistakes. Sorry for that.

The PR was implemented according to my previous experiences and inclinations. I forgot to ask what kind of solutions the community expected. Sorry.

I would like to know:

  • if I should have used docker-compose;
  • if I shouldn't have started the server in the build.

If I forgot something, please let me know.

Thanks for you attention.

@mansogf

mansogf commented Aug 23, 2021

Copy link
Copy Markdown
Contributor

Hi @mansogf,

Looking at the context (contribution to a community), I realized that I made some mistakes. Sorry for that.

The PR was implemented according to my previous experiences and inclinations. I forgot to ask what kind of solutions the community expected. Sorry.

I would like to know:

  • if I should have used docker-compose;
  • if I shouldn't have started the server in the build.

If I forgot something, please let me know.

Thanks for you attention.

The answers are yes and yes =) .

@m-theus m-theus changed the title Fix #2 - Setup CI/CD [WIP] Fix #2 - Setup CI/CD Aug 25, 2021
@m-theus m-theus force-pushed the 2-setup-ci/cd-for-backend branch 3 times, most recently from ad30cb9 to 3b74756 Compare August 25, 2021 23:33
Fix Computer-Progress#2: Add build badge to README.md

Fix Computer-Progress#2: Add pytest-cov

Fix Computer-Progress#2: Add .coveragerc

Fix Computer-Progress#2: Add flake8

Fix Computer-Progress#2: Add test script

Fix Computer-Progress#2: Add workflow to test

Fix Computer-Progress#2: Add workflow to style

Fix Computer-Progress#2: Update README.md and pyproject.toml

Fix Computer-Progress#2: Update README.md

Fix Computer-Progress#2: Adapting solution to the target repository
@m-theus m-theus force-pushed the 2-setup-ci/cd-for-backend branch from 3b74756 to 1fef69a Compare August 25, 2021 23:40
@m-theus m-theus changed the title [WIP] Fix #2 - Setup CI/CD Fix #2 - Setup CI/CD Aug 26, 2021
@m-theus

m-theus commented Aug 26, 2021

Copy link
Copy Markdown
Author

@mansogf, Changes completed.

@m-theus m-theus force-pushed the 2-setup-ci/cd-for-backend branch from 765407b to cb83519 Compare September 8, 2021 17:41
@m-theus m-theus force-pushed the 2-setup-ci/cd-for-backend branch from 9a0aed5 to 34764b7 Compare September 8, 2021 18:30
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