Skip to content

Mark some tests as long #929

Merged
merged 5 commits into from Sep 8, 2022
Merged

Mark some tests as long #929

merged 5 commits into from Sep 8, 2022

Conversation

Mr-Geekman
Copy link
Contributor

@Mr-Geekman Mr-Geekman commented Sep 7, 2022 •
edited

Before submitting (must do checklist)

  • Did you read the contribution guide?
  • Did you update the docs? We use Numpy format for all the methods and classes.
  • Did you write any new necessary tests?
  • Did you update the CHANGELOG?

Proposed Changes

Look #885.

Closing issues

Closes #885.

@Mr-Geekman Mr-Geekman self-assigned this Sep 7, 2022
@github-actions
Copy link

github-actions bot commented Sep 7, 2022 •
edited

🚀 Deployed on https://deploy-preview-929--etna-docs.netlify.app

@github-actions github-actions bot temporarily deployed to pull request September 7, 2022 16:52 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 7, 2022 17:27 Inactive
@codecov-commenter
Copy link

codecov-commenter commented Sep 7, 2022 •
edited

Codecov Report

Merging #929 (069ea56) into master (533fce8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #929   +/-   ##
=======================================
  Coverage   84.99%   84.99%           
=======================================
  Files         133      133           
  Lines        7611     7611           
=======================================
  Hits         6469     6469           
  Misses       1142     1142

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@martins0n martins0n self-requested a review September 8, 2022 07:22
@github-actions github-actions bot temporarily deployed to pull request September 8, 2022 12:46 Inactive
martins0n
martins0n suggested changes Sep 8, 2022
View reviewed changes
Copy link
Contributor

@martins0n martins0n left a comment

Choose a reason for hiding this comment

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

Let's make 2 long queues.
Current long mark is already "overcommitted"

@github-actions github-actions bot temporarily deployed to pull request September 8, 2022 15:08 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 8, 2022 15:15 Inactive
martins0n
martins0n approved these changes Sep 8, 2022
View reviewed changes
Copy link
Contributor

@martins0n martins0n left a comment

Choose a reason for hiding this comment

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

👍

@martins0n martins0n merged commit 9aede87 into master Sep 8, 2022
@martins0n martins0n deleted the issue-885 branch September 8, 2022 16:55
@Mr-Geekman Mr-Geekman mentioned this pull request Sep 9, 2022
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@martins0n martins0n martins0n approved these changes

Assignees

@Mr-Geekman Mr-Geekman

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Mark some tests as long
3 participants
@Mr-Geekman @codecov-commenter @martins0n