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

Arch rules & build update #2850

Merged
merged 2 commits into from
Nov 18, 2023
Merged

Arch rules & build update #2850

merged 2 commits into from
Nov 18, 2023

Conversation

amolenaar
Copy link
Member

PR Type

What kind of change does this PR introduce?

  • Bug fix
  • Feature
  • Chore (refactoring, formatting, local variables, other cleanup)
  • Documentation content changes

What is the new behavior?

  • Architecture test rules have been fine tuned. gaphor.diagram is still a bit problematic since it has a (cyclic) dependency on gaphor.ui and gaphor.UML, but the dependencies have been explicitly specified now.
  • Build artifacts also for PR's created by Dependabot. We should be able to assess updates on libraries like PyInstaller.

Add notable exclusion for file dialog and error handler.
If we get updates on libraries like PyInstaller, the build should
tell us if it works.
@danyeaw danyeaw added the chore Maintenance related PR label Nov 18, 2023
Copy link
Member

@danyeaw danyeaw left a comment

Choose a reason for hiding this comment

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

Looks good, thanks @amolenaar!

@danyeaw danyeaw merged commit 425ec99 into main Nov 18, 2023
32 checks passed
@danyeaw danyeaw deleted the arch-rules branch November 18, 2023 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Maintenance related PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants