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

feat: Installation access tokens for Git operations on GH PLUTO-772 #1931

Merged
merged 48 commits into from
Jan 9, 2024

Conversation

claudiacarpinteiro
Copy link
Contributor

@claudiacarpinteiro claudiacarpinteiro commented Nov 9, 2023

For GitHub organizations, the Codacy GitHub App will start authenticating as an app installation and using installation access tokens for Git operations. SSH keys will only be used as a fallback mechanism.

This pull request updates the documentation to consider this product change.

👀 Live preview

🚧 To do

  • Update content considering the new behavior
  • Request validation
  • Apply feedback
  • Wait for feature release
  • Remove don't merge label before merging

@claudiacarpinteiro claudiacarpinteiro self-assigned this Nov 9, 2023
Copy link
Contributor

github-actions bot commented Nov 9, 2023

Overall readability score: 54.23 (🔴 -0.03)

File Readability
we-no-longer-have-access-to-this-repository.md 59.49 (🔴 -1.15)
which-permissions-does-codacy-need-from-my-account.md 57 (🔴 -2.5)
removing-your-repository.md 43.21 (🔴 -2.58)
using-submodules.md 59.85 (🔴 -0.25)
View detailed metrics

🟢 - Shows an increase in readability
🔴 - Shows a decrease in readability

File Readability FRE GF ARI CLI DCRS
we-no-longer-have-access-to-this-repository.md 59.49 36.39 10.38 11.6 11.25 7.06
  🔴 -1.15 🔴 -0.3 🔴 -0.21 🔴 -0.3 🔴 -0.17 🔴 -0.02
which-permissions-does-codacy-need-from-my-account.md 57 35.68 10.84 12 11.78 7.14
  🔴 -2.5 🔴 -8.05 🔴 -0.12 🔴 -0.1 🔴 -0.35 🔴 -0.1
removing-your-repository.md 43.21 9.59 11.86 14.7 14.39 7.18
  🔴 -2.58 🔴 -7.95 🔴 -0.24 🔴 -0.1 🔴 -0.41 🔴 -0.04
using-submodules.md 59.85 37.3 9.65 12.2 12.35 6.63
  🔴 -0.25 🟢 +0.1 🟢 +0.23 🔴 -0.4 🔴 -0.4 🟢 +0.15

Averages:

  Readability FRE GF ARI CLI DCRS
Average 54.23 43.09 10.86 12.37 12.53 7.87
  🔴 -0.03 🔴 -0.08 🟢 +0 🟢 +0 🔴 -0.01 🟢 +0
View metric targets
Metric Range Ideal score
Flesch Reading Ease 100 (very easy read) to 0 (extremely difficult read) 60
Gunning Fog 6 (very easy read) to 17 (extremely difficult read) 8 or less
Auto. Read. Index 6 (very easy read) to 14 (extremely difficult read) 8 or less
Coleman Liau Index 6 (very easy read) to 17 (extremely difficult read) 8 or less
Dale-Chall Readability 4.9 (very easy read) to 9.9 (extremely difficult read) 6.9 or less

@claudiacarpinteiro claudiacarpinteiro added the don't merge Don't merge this pull request yet label Nov 9, 2023
@github-actions github-actions bot temporarily deployed to Netlify November 9, 2023 12:00 Inactive
@github-actions github-actions bot temporarily deployed to Netlify November 9, 2023 12:40 Inactive
@github-actions github-actions bot temporarily deployed to Netlify November 9, 2023 16:08 Inactive
@github-actions github-actions bot temporarily deployed to Netlify November 9, 2023 16:13 Inactive
@github-actions github-actions bot temporarily deployed to Netlify November 9, 2023 16:15 Inactive
@github-actions github-actions bot temporarily deployed to Netlify November 9, 2023 17:46 Inactive
@github-actions github-actions bot temporarily deployed to Netlify November 9, 2023 17:49 Inactive
@github-actions github-actions bot temporarily deployed to Netlify November 9, 2023 17:50 Inactive
@github-actions github-actions bot temporarily deployed to Netlify November 9, 2023 18:05 Inactive
@github-actions github-actions bot temporarily deployed to Netlify November 9, 2023 18:10 Inactive
@claudiacarpinteiro claudiacarpinteiro marked this pull request as ready for review November 9, 2023 18:14
@claudiacarpinteiro claudiacarpinteiro requested a review from a team as a code owner November 9, 2023 18:14
@github-actions github-actions bot temporarily deployed to Netlify December 15, 2023 11:09 Inactive
Co-authored-by: Nicola Klemenc <[email protected]>
@github-actions github-actions bot temporarily deployed to Netlify December 15, 2023 11:35 Inactive
@github-actions github-actions bot temporarily deployed to Netlify December 15, 2023 15:58 Inactive
@github-actions github-actions bot temporarily deployed to Netlify December 26, 2023 12:07 Inactive
@github-actions github-actions bot temporarily deployed to Netlify December 27, 2023 16:47 Inactive
@github-actions github-actions bot temporarily deployed to Netlify January 4, 2024 15:47 Inactive
@github-actions github-actions bot temporarily deployed to Netlify January 4, 2024 17:10 Inactive
@github-actions github-actions bot temporarily deployed to Netlify January 4, 2024 17:22 Inactive
@claudiacarpinteiro
Copy link
Contributor Author

@jorgebraz Thank you for the input. I incorporated your feedback. 🙏

@github-actions github-actions bot temporarily deployed to Netlify January 5, 2024 17:31 Inactive
@github-actions github-actions bot temporarily deployed to Netlify January 5, 2024 17:43 Inactive
@github-actions github-actions bot temporarily deployed to Netlify January 8, 2024 10:36 Inactive
@github-actions github-actions bot temporarily deployed to Netlify January 9, 2024 15:15 Inactive
@claudiacarpinteiro claudiacarpinteiro removed the don't merge Don't merge this pull request yet label Jan 9, 2024
@claudiacarpinteiro claudiacarpinteiro merged commit 1cdeb00 into master Jan 9, 2024
7 of 8 checks passed
@claudiacarpinteiro claudiacarpinteiro deleted the PLUTO-772-gh-installation-access-tokens branch January 9, 2024 15:18
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.

5 participants