Skip to content

Conversation

@heyealex
Copy link
Contributor

Submission Checklist

  • Have you installed and run this change against pre-commit? (pre-commit install)
  • Are all tests passing? (make tests)
  • Have you written unit tests to cover this change?
  • Is unit test coverage still above 80%?
  • Have you updated all applicable documentation?
  • Have you followed the guidelines in our Contributing document?

cboneti
cboneti previously approved these changes Jun 24, 2022
@heyealex
Copy link
Contributor Author

Had to push a fix, so the initial review got dismissed. Tested locally and in the integration tests and all is working now.

@nick-stroud nick-stroud assigned heyealex and unassigned nick-stroud Jun 25, 2022
@heyealex heyealex merged commit e70963e into GoogleCloudPlatform:develop Jun 25, 2022
@heyealex heyealex deleted the omnia-slurm-user-setting branch June 25, 2022 01:07
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.

3 participants