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

Upgrade CodeSee workflow to version 2 #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ghost
Copy link

@ghost ghost commented Dec 5, 2022

CodeSee is a code visibility platform.

This change updates the CodeSee workflow file to the latest version for security, maintenance, and support improvements (see changelog below).

That workflow file:

  • runs CodeSee's code analysis on every PR push and merge
  • uploads that analysis to CodeSee.
  • It does not transmit your code.

The code analysis is used to generate maps and insights about this codebase.

CodeSee workflow changelog:

  • Improved security: Updates permission to be read-only.
  • Improved future maintenance: Replaces the body of the workflow with a single github action: codesee-action. This makes it significantly easier for CodeSee to introduce future improvements and fixes without requiring another PR like this.
  • Improved Python support: The action now properly supports Python 3.11, and will continue to support new Python versions as they are released.

@ghost
Copy link
Author

ghost commented Dec 5, 2022

👇 Click on the image for a new way to code review
  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

@codecov
Copy link

codecov bot commented Dec 5, 2022

Codecov Report

Base: 83.11% // Head: 80.81% // Decreases project coverage by -2.29% ⚠️

Coverage data is based on head (74c5914) compared to base (5800012).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
- Coverage   83.11%   80.81%   -2.30%     
==========================================
  Files          28       19       -9     
  Lines        1753     1569     -184     
  Branches     1569     1569              
==========================================
- Hits         1457     1268     -189     
- Misses        296      301       +5     
Impacted Files Coverage Δ
src/subcommand/add_env.rs 81.51% <0.00%> (-2.53%) ⬇️
src/subcommand/remove.rs 82.35% <0.00%> (-1.97%) ⬇️
rualdlib/src/lib.rs 79.58% <0.00%> (-0.27%) ⬇️
tests/rualdi/common.rs 100.00% <0.00%> (ø)
tests/rualdi/resolve.rs
tests/rualdi/remove_env.rs
tests/rualdi/list_env.rs
tests/rualdi/main.rs
tests/rualdi/remove.rs
tests/rualdi/resolve_env.rs
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

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.

0 participants