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

[GEN-1191] Add file logging #552

Merged
merged 5 commits into from
Mar 13, 2024
Merged

[GEN-1191] Add file logging #552

merged 5 commits into from
Mar 13, 2024

Conversation

rxu17
Copy link
Contributor

@rxu17 rxu17 commented Mar 9, 2024

Purpose: This small PR adds specific file logging when getting all of the center input files for validation/processing. See JIRA ticket for more details but in the past we had run into issues during this step and knowing the exact file prior to the retrieving of the file is helpful.

Testing:

  • Tested locally
  • Tested on nextflow workflow

@rxu17 rxu17 requested a review from a team as a code owner March 9, 2024 04:40
Copy link

sonarqubecloud bot commented Mar 9, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

@danlu1 danlu1 left a comment

Choose a reason for hiding this comment

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

LGTM

@rxu17 rxu17 merged commit 9c10710 into develop Mar 13, 2024
8 checks passed
@rxu17 rxu17 deleted the gen-1191-add-file-logging branch March 13, 2024 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants