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

core: fix vendored path config for benchmark files #333

Merged
merged 1 commit into from
Feb 1, 2022

Conversation

madmike200590
Copy link
Collaborator

This is a leftover from PR #274: The path benchmarks/**, containing an enormous amount of ASP sources, should be marked as vendored, i.e. excluded from github statistics. Since the folder in question got moved into alpha-core, we need to adapt the config in .gitattributes accordingly.

@codecov
Copy link

codecov bot commented Feb 1, 2022

Codecov Report

Merging #333 (96be6e1) into master (d42818a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #333   +/-   ##
=========================================
  Coverage     70.23%   70.23%           
  Complexity     2123     2123           
=========================================
  Files           182      182           
  Lines          8023     8023           
  Branches       1423     1423           
=========================================
  Hits           5635     5635           
  Misses         2028     2028           
  Partials        360      360           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d42818a...96be6e1. Read the comment docs.

@madmike200590 madmike200590 merged commit 3ed0118 into master Feb 1, 2022
@madmike200590 madmike200590 deleted the fix-vendored-paths branch February 1, 2022 17:45
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