Skip to content

Conversation

@mdwyman
Copy link
Contributor

@mdwyman mdwyman commented May 7, 2025

Adding support for serial-based XIA PFCU filters; Keyence CL3000 distance sensor; Thorlabs SC-10 shutter controller; and Uniblitz shutter controllers.

.loglogin Outdated
Copy link
Member

Choose a reason for hiding this comment

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

This file should be deleted. It is created by scripts that are automatically run on APS linux machines.

Copy link
Contributor

Choose a reason for hiding this comment

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

Common to add .loglogin to the repostory's .gitignore file. GitHub makes it possible to provide an organization-wide set of features (workflows, for example). The Bluesky team has been discussing adding a org-level .gitignore (in the BCDA-APS org) that would include .DS_Store, .loglogin, and some other common things one would not normally keep in a repo (such as .vscode).

Copy link
Member

@kmpeters kmpeters May 7, 2025

Choose a reason for hiding this comment

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

Change this:

iocsh/

To this:

/iocsh/

So that only the top-level, autogenerated iocsh dir is ignored.

Copy link
Member

@kmpeters kmpeters left a comment

Choose a reason for hiding this comment

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

I'm comfortable merging this pull request.

@kmpeters
Copy link
Member

kmpeters commented May 7, 2025

@prjemian are there any other changes you think should be made before this PR is merged?

Copy link
Contributor

@prjemian prjemian left a comment

Choose a reason for hiding this comment

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

Changes in .gitignore look right. I'm skipping review of the other changes.

@mdwyman mdwyman merged commit e052e29 into master May 7, 2025
0 of 6 checks passed
@mdwyman mdwyman deleted the xiapfcu branch May 7, 2025 22:08
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.

4 participants