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

[DEFECT] General RAVEN Maintenance Updates #1806

Open
8 of 13 tasks
dylanjm opened this issue Apr 7, 2022 · 0 comments
Open
8 of 13 tasks

[DEFECT] General RAVEN Maintenance Updates #1806

dylanjm opened this issue Apr 7, 2022 · 0 comments
Labels
defect priority_normal task This tag should be used for any new capability, improvement or enanchment

Comments

@dylanjm
Copy link
Collaborator

dylanjm commented Apr 7, 2022

Thank you for the defect report

Defect Description

This issue can be used in reference to PRs that only make minor, non-functional changes to RAVEN's code. The expectation would be that these changes are small, do not affect any developer or user functionality, and are independent of other PRs.

A few examples of changes that would apply to this issue:

  • Fixing SyntaxWarnings emitted during RAVEN runtime
  • Fixing deprecation warnings from dependent library APIs
  • Modifying a comment
  • Raising/lowering the level of output to stdout (debug, warning, all, message, etc)
  • Removing commented code
  • Removing inconsequential code

Steps to Reproduce

The fix should be properly explained in any PR referencing this issue.

Expected Behavior

I imagine a good rule of thumb is that nobody would notice these changes took place.

Screenshots and Input Files

No response

OS

Linux

OS Version

No response

Dependency Manager

CONDA

For Change Control Board: Issue Review

  • Is it tagged with a type: defect or task?
  • Is it tagged with a priority: critical, normal or minor?
  • If it will impact requirements or requirements tests, is it tagged with requirements?
  • If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
  • Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure

  • If the issue is a defect, is the defect fixed?
  • If the issue is a defect, is the defect tested for in the regression test system? (If not explain why not.)
  • If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
  • If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
  • If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?
@dylanjm dylanjm added priority_normal task This tag should be used for any new capability, improvement or enanchment defect labels Apr 7, 2022
@dylanjm dylanjm pinned this issue Apr 7, 2022
@joshua-cogliati-inl joshua-cogliati-inl mentioned this issue Jun 13, 2022
9 tasks
@wangcj05 wangcj05 mentioned this issue Sep 1, 2022
9 tasks
@wangcj05 wangcj05 mentioned this issue Nov 10, 2022
9 tasks
@wangcj05 wangcj05 mentioned this issue Nov 16, 2022
9 tasks
This was referenced Apr 28, 2023
This was referenced Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect priority_normal task This tag should be used for any new capability, improvement or enanchment
Projects
None yet
Development

No branches or pull requests

1 participant