Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Refactor using package com.clp.project #1

Merged
merged 10 commits into from
Jun 4, 2024
Merged

Refactor using package com.clp.project #1

merged 10 commits into from
Jun 4, 2024

Conversation

boozec
Copy link
Owner

@boozec boozec commented Jun 4, 2024 •
edited
Loading

I dunno if this name works for you but who cares.

Also, added a CI to check the make all command for each commit.

@gabrielegenovese
Copy link
Collaborator

worka for me

gabrielegenovese
gabrielegenovese requested changes Jun 4, 2024
View reviewed changes
Copy link
Collaborator

@gabrielegenovese gabrielegenovese left a comment •
edited
Loading

Choose a reason for hiding this comment

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

non funziona il comando make runall

java -cp lib/antlr-4.13.1-complete.jar:out ParseAll 
Error: Could not find or load main class ParseAll
Caused by: java.lang.ClassNotFoundException: ParseAll
make: *** [Makefile:24: runall] Error 1

@gabrielegenovese gabrielegenovese requested review from L0P0P and removed request for L0P0P June 4, 2024 11:07
@boozec boozec merged commit 1c87619 into main Jun 4, 2024
2 checks passed
@boozec boozec deleted the refactor-package branch June 4, 2024 12:11
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@gabrielegenovese gabrielegenovese gabrielegenovese approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@boozec @gabrielegenovese