Skip to content

feat: Create a GitHub action to run Codacy's CLI [CY-2806] #295

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.

Sign up for GitHub

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

Jump to bottom
Merged
afsmeira merged 2 commits into from
Sep 18, 2020
Merged

feat: Create a GitHub action to run Codacy's CLI [CY-2806] #295

afsmeira merged 2 commits into from
Sep 18, 2020

Conversation

@afsmeira
Copy link
Contributor

@afsmeira afsmeira commented Sep 16, 2020

No description provided.

afsmeira
afsmeira commented Sep 16, 2020
View reviewed changes
afsmeira
afsmeira commented Sep 16, 2020
View reviewed changes
@afsmeira afsmeira force-pushed the feat/codacy-cli-action-CY2806 branch 21 times, most recently from 43c4831 to 91e62c4 Compare September 16, 2020 16:16
@afsmeira afsmeira force-pushed the feat/codacy-cli-action-CY2806 branch 13 times, most recently from 99e421d to 8a64e53 Compare September 17, 2020 11:35
rtfpessoa
rtfpessoa suggested changes Sep 17, 2020
View reviewed changes
@rtfpessoa
Copy link
Contributor

rtfpessoa commented Sep 17, 2020

@afsmeira new cli release is done here https://app.circleci.com/pipelines/github/codacy/codacy-analysis-cli/686/workflows/ac11faab-7153-4c5c-a8b6-0e81fdd6cd8d

@afsmeira afsmeira force-pushed the feat/codacy-cli-action-CY2806 branch from 8cf7f1f to 6a34606 Compare September 18, 2020 08:44
afsmeira
afsmeira commented Sep 18, 2020
View reviewed changes
@afsmeira
clean: Address review comments
56931a7 
- Remove default values
- Add description to input parameters
- Add action author
- Improve documentation
@afsmeira afsmeira force-pushed the feat/codacy-cli-action-CY2806 branch from 6a34606 to 56931a7 Compare September 18, 2020 10:04
@afsmeira afsmeira merged commit f9640d9 into master Sep 18, 2020
@delete-merged-branch delete-merged-branch bot deleted the feat/codacy-cli-action-CY2806 branch September 18, 2020 10:20
rtfpessoa added a commit that referenced this pull request Sep 18, 2020
@rtfpessoa
Revert "Merge pull request #295 from codacy/feat/codacy-cli-action-CY…
f3a8241 
…2806"

This reverts commit f9640d9, reversing
changes made to 85d779e.
rtfpessoa added a commit that referenced this pull request Sep 18, 2020
@rtfpessoa
Merge pull request #298 from codacy/revert-action
1ac8795 
Revert "Merge pull request #295 from codacy/feat/codacy-cli-action-CY…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gjsduarte gjsduarte Awaiting requested review from gjsduarte

@andreaTP andreaTP Awaiting requested review from andreaTP

@DReigada DReigada Awaiting requested review from DReigada

@lolgab lolgab Awaiting requested review from lolgab

@pedrocodacy pedrocodacy Awaiting requested review from pedrocodacy

2 more reviewers

@rtfpessoa rtfpessoa rtfpessoa approved these changes

@jllopes jllopes jllopes approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@afsmeira @rtfpessoa @jllopes