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

EAGLE-1137: Updated some attributes in the modelData part of the output palette #252

Merged
merged 5 commits into from
Jul 9, 2024

Conversation

james-strauss-uwa
Copy link
Collaborator

@james-strauss-uwa james-strauss-uwa commented May 15, 2024

Changed eagleVersion -> generatorVersion, eagleCommitHash -> generatorCommitHash, added generatorName.

…Changed eagleVersion -> generatorVersion, eagleCommitHash -> generatorCommitHash, added generatorName.
@james-strauss-uwa james-strauss-uwa self-assigned this May 15, 2024
@james-strauss-uwa james-strauss-uwa changed the title Updated some attributes in the modelData part of teh output palette. … EAGLE-1137: Updated some attributes in the modelData part of the output palette May 31, 2024
@myxie
Copy link
Collaborator

myxie commented Jun 25, 2024

@james-strauss-uwa it looks like the merge conflicts are due to missing commits from #260? If you pull that locally you may have to re-apply some of the generator changes made in this PR to the new graphs (apologies).

@coveralls
Copy link

Coverage Status

coverage: 79.479%. remained the same
when pulling 1c1145f on eagle-1137
into d2262f4 on master.

@coveralls
Copy link

Coverage Status

coverage: 79.479%. remained the same
when pulling 1667ca6 on eagle-1137
into d2262f4 on master.

@james-strauss-uwa
Copy link
Collaborator Author

One check failed in the testing above, but succeeded after re-running.

The merge was easy, all automatic.

Copy link
Collaborator

@myxie myxie left a comment

Choose a reason for hiding this comment

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

Sorry I missed re-reviewing this one! I'll merge it as well because I need these changes. :')

@myxie myxie merged commit f2a1e7f into master Jul 9, 2024
20 of 21 checks passed
This pull request was closed.
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