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

a small change to test Jenkins #1357

Closed
wants to merge 2 commits into from
Closed

a small change to test Jenkins #1357

wants to merge 2 commits into from

Conversation

mer-a-o
Copy link

@mer-a-o mer-a-o commented Jan 6, 2021

The first line should be a single-line "purpose" for this change

TYPE: choose one of [bug fix, enhancement, new feature, feature removed, no impact, text only]

KEYWORDS: 5 to 10 words related to commit, separated by commas

SOURCE: Either "developer's name (affiliation)" .XOR. "internal" for a WRF Dev committee member

DESCRIPTION OF CHANGES:
Problem:
Generally or specifically, what was wrong and needed to be addressed?

Solution:
What was down algorithmically and in the source code to address the problem?

ISSUE: For use when this PR closes an issue.
Fixes #123

LIST OF MODIFIED FILES: list of changed files (use git diff --name-status master to get formatted list)

TESTS CONDUCTED:

  1. Do mods fix problem? How can that be demonstrated, and was that test conducted?
  2. Are the Jenkins tests all passing?

RELEASE NOTE: Include a stand-alone message suitable for the inclusion in the minor and annual releases. A publication citation is appropriate.

@mer-a-o mer-a-o requested a review from a team as a code owner January 6, 2021 04:20
@davegill
Copy link
Contributor

davegill commented Jan 7, 2021

@mer-a-o
Maryam,
Are you getting the jenkins tests?

@mer-a-o
Copy link
Author

mer-a-o commented Jan 7, 2021

@davegill I received an email from Jenkins after pushing the last commit this morning.

@davegill davegill closed this Jan 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants