Skip to content

Update manufacturer names - #1832

Merged
peternewman merged 3 commits into
OpenLightingProject:masterfrom
peternewman:e1.31
Mar 8, 2023
Merged

Update manufacturer names#1832
peternewman merged 3 commits into
OpenLightingProject:masterfrom
peternewman:e1.31

Conversation

@peternewman

Copy link
Copy Markdown
Member

No description provided.

@peternewman
peternewman requested a review from kripton March 6, 2023 18:00

@kripton kripton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

phew, a PR without description ;)
codespell complains about some but changes look good as they are

@@ -630,10 +1066,22 @@ manufacturer {
manufacturer_id: 20781
manufacturer_name: "DJPOWER ELECTRONIC STAGE LIGHTING FIXTURE FACTORY (GUANGZHOU)"
}
manufacturer {
manufacturer_id: 434
manufacturer_name: "DJSI Schinstad ANS (Northern Light)"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we codespell-ignore those somehow?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah I forgot to revisit that, now done and will hopefully come out clean...

@peternewman
peternewman enabled auto-merge March 8, 2023 00:08
@peternewman peternewman added this to the 0.11.0 milestone Mar 8, 2023
@peternewman
peternewman merged commit 5e4aa92 into OpenLightingProject:master Mar 8, 2023
@peternewman peternewman mentioned this pull request Jul 12, 2026
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