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.

Sign up for GitHub

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

Jump to bottom

Fixing compilation for most up to date RIDDL version #73

Merged
merged 24 commits into from
Mar 9, 2024
Merged

Fixing compilation for most up to date RIDDL version #73

merged 24 commits into from
Mar 9, 2024

Conversation

AlexWeinstein92
Copy link
Contributor

@AlexWeinstein92 AlexWeinstein92 commented Mar 5, 2024 •
edited
Loading

Remaining errror at time of PR creation:

  • State, aggregate, handler/event comments (bug in RIDDL)
  • unsure whether to delete interaction (reservationStories)
  • LocationId has the following error everywhere it's used: Expected one of (end-of-input | whitespace after keyword)

AlexWeinstein92 and others added 23 commits March 5, 2024 17:39
@AlexWeinstein92
removing unnecessary comments
6a903e0
@AlexWeinstein92
removing unnecessary comments
9a3428b
@AlexWeinstein92
fixing after RIDDL update for comments bug
49bd5c2
@AlexWeinstein92
commenting out interactions
1874a9a
@AlexWeinstein92
commenting out examples, fixing bugs accordingly
7b84c80
@reid-spencer
Modernize the build
337b39c
@reid-spencer
Fix options and LocationId errors
f24280a
@reid-spencer
Fix remaining parsing errors
34d86ec
@AlexWeinstein92
various changes
c80d666
@AlexWeinstein92
Merge remote-tracking branch 'origin/Update_For_RIDDL_Udpates' into U…
3b66694 
…pdate_For_RIDDL_Udpates

# Conflicts:
#	src/main/riddl/Commerce/commerce.riddl
#	src/main/riddl/Events/Reservations/reservationTypeDefinitions.riddl
#	src/main/riddl/Gateway/MemberGateway/memberGateway.riddl
#	src/main/riddl/Organization/organization.riddl
#	src/main/riddl/Store/Store.riddl
#	src/main/riddl/Tenant/tenant.riddl
#	src/main/riddl/Venues/Locations/locationMessages.riddl
#	src/main/riddl/Venues/Locations/locationTypeDefinitions.riddl
#	src/main/riddl/Venues/venueLocationsProjections.riddl
#	src/main/riddl/Venues/venueLocationsReservationsMessages.riddl
#	src/main/riddl/Venues/venueMessages.riddl
@reid-spencer
Enable the group-messages-by-kind option
b393bb0
@reid-spencer
Merge remote-tracking branch 'origin/Update_For_RIDDL_Udpates' into U…
7dff9a3 
…pdate_For_RIDDL_Udpates
@AlexWeinstein92
lots of error fixes
dfd3b0f
@reid-spencer
Fix some validation errors
0d987b0
@reid-spencer
Fix various undefined references and ambiguities
b3ea98c
@AlexWeinstein92
applicatioon & organization fixes
f5f058a
@AlexWeinstein92
Merge remote-tracking branch 'origin/Update_For_RIDDL_Udpates' into U…
a406ccb 
…pdate_For_RIDDL_Udpates
@AlexWeinstein92
event, order, reservation, tenant fixes
4b8d118
@AlexWeinstein92
store fixes
2b4234b
@AlexWeinstein92
product, event fixes
e8f3dab
@AlexWeinstein92
no errors left
3e8ced2
@reid-spencer
Correct RIDDL input to resolve remaining issues per riddlc 0.39
b6d2438
@reid-spencer
Upgrade to riddlc 0.39.0 release
11aee62
@reid-spencer reid-spencer merged commit b1294b4 into main Mar 9, 2024
2 checks passed
@reid-spencer reid-spencer deleted the Update_For_RIDDL_Udpates branch March 9, 2024 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JamesYopp JamesYopp Awaiting requested review from JamesYopp JamesYopp is a code owner

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
@AlexWeinstein92 @reid-spencer