BUG: check-warnings is reporting warnings from `test` set when building running PRs (for other actions)

Open
#207 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
35/100
Issue type
Bug
Clarity
Needs clarification
Activity status
Stale
Tech stack
github-actions
Domain
ci-cd, testing-qa

Research direction

Start by examining the check-warnings tests and how the tests are run for actions such as weekly-report, paying attention to whether the default target path includes the test set. Done means warnings intentionally present in the test set are excluded from check-warnings reports when other actions run on pull requests.

Written by the indexing model from the issue text.

Description

The check-warnings tests are reporting the Warnings found in the test set when opening PRs

Given they are meant to be there for the sake of testing, we need to fix this.

I will need to see how the tests are run and then exclude them when the action is running (as it is intended). Perhaps the default target path is including the test set?

Here is an example of the report when editing the weekly-report action

Image
Dominant language
No language data
Stars
0
Forks
0
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from QuantEcon/meta

All issues in QuantEcon/meta

Similar issues

More DevOps issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.