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

Deploy Branch Status (October / November 2022) #35

Merged
merged 31 commits into from
Dec 1, 2022
Merged

Deploy Branch Status (October / November 2022) #35

merged 31 commits into from
Dec 1, 2022

Conversation

klauer
Copy link
Contributor

@klauer klauer commented Oct 7, 2022

*** Not to be merged during October ***

This PR can be used to easily see when cron-based pushes to GitHub happen, and allow us an opportunity to review changes prior to merging into master.

(We still have not addressed #33 and we should before the end of this month)

@klauer
Copy link
Contributor Author

klauer commented Oct 18, 2022

@klauer klauer changed the title Deploy Branch Status (October 2022) Deploy Branch Status (October / November 2022) Nov 9, 2022
@klauer
Copy link
Contributor Author

klauer commented Dec 1, 2022

Here we are in December but we still rely on Tyler's cron job. Maybe 2023?

Edit: and diff is too large to review :(

@klauer klauer marked this pull request as ready for review December 1, 2022 17:16
@tangkong
Copy link

tangkong commented Dec 1, 2022

Comparing to my device_config, the major changes seem to be

  • leviton device changes
  • Lightpath device modifications

lots of devices got moved around so the diff is also unintelligible locally. :(

@ZLLentz
Copy link
Member

ZLLentz commented Dec 1, 2022

lots of devices got moved around so the diff is also unintelligible locally

That's unfortunate. I'll take a peek locally as well but I suspect everything is fine and this is just to see if there's anyone we need to talk to.

@ZLLentz
Copy link
Member

ZLLentz commented Dec 1, 2022

I'm going to do the merge now in advance of the next push and bring "figure out how to securely set up fine-grained PATs to set up this sync using one of our users" up higher in my to-do list

@ZLLentz ZLLentz merged commit bc7a814 into master Dec 1, 2022
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