Skip to content

Create rust-clippy.yml#8

Draft
victoryforphil wants to merge 1 commit into
mainfrom
victoryforphil-patch-1
Draft

Create rust-clippy.yml#8
victoryforphil wants to merge 1 commit into
mainfrom
victoryforphil-patch-1

Conversation

@victoryforphil

Copy link
Copy Markdown
Owner

No description provided.

@codecov

codecov Bot commented Nov 11, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c1cef59) 93.80% compared to head (2cd9f15) 93.80%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage   93.80%   93.80%           
=======================================
  Files           9        9           
  Lines         210      210           
=======================================
  Hits          197      197           
  Misses         13       13           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-advanced-security

Copy link
Copy Markdown

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

# The branches below must be a subset of the branches above
branches: [ "main" ]
schedule:
- cron: '37 12 * * 1'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Feels like this should just be done on commit to main through a merge. Is this github being sneaky about credit usage ?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

You're correct, this was a super old PR so not sure of the context of this cron, seems out of place. Not yet merging this since Clippy is a bit much when trying to move quick.

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.

3 participants