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

Update data_classes.py #265

Merged

Conversation

tcmcclan-ornl
Copy link
Contributor

@tcmcclan-ornl tcmcclan-ornl commented Aug 6, 2024

Fixed call to undefined variable.

Description

I failed to change the TypeError message when I updated the routine. I removed the call to the undefined variable "entry" and put in the call to the proper variable.

Fixes issue

Checklist

  • I'm making a PR from a feature branch on my fork into GEOUNED-org/GEOUNED/dev branch
  • I have followed PEP8 style guide for Python or run a formatter such as black or ruff format on my code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

Fixed call to undefined variable.
@psauvan psauvan merged commit 795c26a into GEOUNED-org:dev Aug 6, 2024
8 of 9 checks passed
@tcmcclan-ornl tcmcclan-ornl deleted the Fixing-call-to-undefined-variable branch August 6, 2024 15:23
psauvan added a commit that referenced this pull request Aug 6, 2024
psauvan added a commit that referenced this pull request Aug 6, 2024
psauvan added a commit that referenced this pull request Aug 6, 2024
fix bad black format of PR #265 and add small modifications
psauvan added a commit that referenced this pull request Aug 6, 2024
* Update data_classes.py

fix bad black format of PR #265 and add small modifications

* Update data_classes.py

black is black
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