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

Jump to bottom

ci: speed up lint workflow with hoisting #648

Merged
merged 4 commits into from Aug 30, 2021
Merged

ci: speed up lint workflow with hoisting #648

merged 4 commits into from Aug 30, 2021

Conversation

YanivD
Copy link
Member

@YanivD YanivD commented Aug 29, 2021 •
edited

Which problem is this PR solving?

Short description of the changes

  • Hoising lint action.

@YanivD YanivD requested a review from a team as a code owner August 29, 2021 10:18
@YanivD YanivD changed the title Lint hoist CI: speed up lint and browser-tests workflows with hoisting Aug 29, 2021
@YanivD YanivD changed the title CI: speed up lint and browser-tests workflows with hoisting ci: speed up lint and browser-tests workflows with hoisting Aug 29, 2021
@YanivD YanivD marked this pull request as draft August 29, 2021 10:27
@YanivD YanivD changed the title ci: speed up lint and browser-tests workflows with hoisting ci: speed up lint workflow with hoisting Aug 29, 2021
@codecov
Copy link

codecov bot commented Aug 29, 2021 •
edited

Codecov Report

Merging #648 (7fd9ab9) into main (c7b32e8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #648   +/-   ##
=======================================
  Coverage   96.68%   96.68%           
=======================================
  Files          13       13           
  Lines         634      634           
  Branches      124      124           
=======================================
  Hits          613      613           
  Misses         21       21

@YanivD YanivD marked this pull request as ready for review August 29, 2021 10:48
@dyladan dyladan merged commit d711127 into open-telemetry:main Aug 30, 2021
@blumamir blumamir mentioned this pull request Aug 31, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vmarchaud vmarchaud vmarchaud approved these changes

@Flarna Flarna Flarna approved these changes

@blumamir blumamir blumamir approved these changes

Assignees
No one assigned
Labels
internal
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@YanivD @vmarchaud @Flarna @blumamir @dyladan