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

T4 update browse dialogs #9

Closed

Conversation

kiritsuku
Copy link

No automatic tests, everything is tested manually.

What do you think, are the file dialogs ok?

Changes:

- Browse/Add and New configuration dialogs can now handle files
  that exist outside of the workspace
- It is no longer possible to add the same configuration multiple
  times to the list of configurations
- Cleanup imports

Fixes scalastyle#4
@matthewfarwell
Copy link
Member

I'll have a look at this when I have a minute. Thanks.

@matthewfarwell
Copy link
Member

In general I like this patch. However, if I have an existing configuration, I'll have to recreate it, and re-assign it to all of my projects. I'll have to think about this a bit more. Maybe checkstyle has the best solution with different types of configuration, inside a project, outside a project etc.

@matthewfarwell
Copy link
Member

Closed. If you still want to do this, please re-open it.

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.

None yet

2 participants