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

GUI initial revision, among other things #59

Merged
merged 11 commits into from
Apr 27, 2024
Merged

GUI initial revision, among other things #59

merged 11 commits into from
Apr 27, 2024

Conversation

mkacct
Copy link
Owner

@mkacct mkacct commented Apr 27, 2024

Primary changes:

  • The first revision of the GUI is now implemented and tested (the framework-independent part is tested)
  • Application behavior shared between CLI and GUI extracted to new class domain.CheckUtil

Minor changes:

  • New class ProductInfo has name and version constants
  • CLI now prints version info
  • JsonFileConfigLoader throws the right kind of exception
  • Renamed directory cli-system-test to just system-test since it is also used by GUI system test now

Things that were not yet developed:

  • The GUI settings page is NYI; the GUI cannot write config yet

@mkacct mkacct changed the title Gui GUI, among other things Apr 27, 2024
@mkacct mkacct changed the title GUI, among other things GUI initial revision, among other things Apr 27, 2024
@mkacct mkacct requested a review from rhit-shirakrk April 27, 2024 20:08
@rhit-shirakrk rhit-shirakrk merged commit 46d8255 into main Apr 27, 2024
1 check passed
@rhit-shirakrk rhit-shirakrk deleted the gui branch April 27, 2024 20:24
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