Skip to content

skip some bad fee cmdline tests and note these aren't currently run in CI #16821

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

Closed
wants to merge 1 commit into from

Conversation

emlowe
Copy link
Contributor

@emlowe emlowe commented Nov 13, 2023

The tests in tests/fee_estimation/cmdline_test.py don't work if run manually due to several issues. These tests currently are not run in CI because the filename doesn't match the expected test_*.py pattern

For now, skipping these tests. This only impacts when you run the tests manually since as noted they aren't run in CI.

@emlowe emlowe added Fixed Required label for PR that categorizes merge commit message as "Fixed" for changelog CI CI changes Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Nov 13, 2023
@emlowe emlowe requested a review from a team as a code owner November 13, 2023 16:47
@emlowe emlowe requested a review from aqk November 13, 2023 16:47
Copy link
Contributor

@altendky altendky left a comment

Choose a reason for hiding this comment

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

Note that this relates somewhat to #16524.

Copy link
Contributor

This PR has been flagged as stale due to no activity for over 60 days. It will not be automatically closed, but it has been given a stale-pr label and should be manually reviewed by the relevant parties.

@github-actions github-actions bot added the stale-pr Flagged as stale and in need of manual review label Dec 29, 2023
@emlowe
Copy link
Contributor Author

emlowe commented Feb 14, 2024

close and reopen

@emlowe emlowe closed this Feb 14, 2024
@emlowe emlowe reopened this Feb 14, 2024
@github-actions github-actions bot removed the stale-pr Flagged as stale and in need of manual review label Feb 15, 2024
Copy link
Contributor

github-actions bot commented Mar 5, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the merge_conflict Branch has conflicts that prevent merge to main label Mar 5, 2024
Copy link
Contributor

This PR has been flagged as stale due to no activity for over 60 days. It will not be automatically closed, but it has been given a stale-pr label and should be manually reviewed by the relevant parties.

@github-actions github-actions bot added the stale-pr Flagged as stale and in need of manual review label Apr 20, 2024
@emlowe emlowe closed this Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI CI changes Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes Fixed Required label for PR that categorizes merge commit message as "Fixed" for changelog merge_conflict Branch has conflicts that prevent merge to main stale-pr Flagged as stale and in need of manual review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants