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

FIX: fixing variables name errors #89

Merged
merged 1 commit into from
Oct 26, 2021
Merged

FIX: fixing variables name errors #89

merged 1 commit into from
Oct 26, 2021

Conversation

JuliethOteroRodriguez
Copy link
Collaborator

@JuliethOteroRodriguez JuliethOteroRodriguez commented Oct 20, 2021

Description
Fixing 2 variable names errors of the script epicsArchChecker that checks epicsArch.txt files.

Motivation and Context
Two variables inside of the script were missing a character. After adding the missing character to the two variables the script was tested.

How Has This Been Tested?

The script was re-tested against a set of epicArch.txt from the hutches XPP and CXI,

How Has This Been Documented?
N/A

Copy link
Contributor

@ZryletTC ZryletTC left a comment

Choose a reason for hiding this comment

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

Assuming all arguments have been tested, should be good to go 👍

@ZryletTC
Copy link
Contributor

I'm just going to merge this and release so it can get used. Future changes can be added to a new PR.

@ZryletTC ZryletTC merged commit 04d55bc into pcdshub:master Oct 26, 2021
@JuliethOteroRodriguez JuliethOteroRodriguez deleted the epicsArchChecker branch December 9, 2021 18:50
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