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

Projections #46

Merged
merged 63 commits into from
Apr 9, 2021
Merged

Projections #46

merged 63 commits into from
Apr 9, 2021

Conversation

jakhog
Copy link
Contributor

@jakhog jakhog commented Apr 8, 2021 •
edited
Loading

Summary

Added support for imperative Projections, and releasing of generated Go code, and some minor cleanup of code generations scripts.

Added

Changed

  • Moved things around in the code generation (Fundamentals+Runtime top level -> Language top level)
  • Removed generated fundamentals code in @dolittle/runtime.contracts, and added dependency
  • Removed Runtime/ Fundamentals/ directories in JS pacakges

Removed

  • Code generation for JavaScript.Web using gRPC web, since we are currently not using it and it was not working.

jakhog and others added 30 commits March 22, 2021 12:49
@jakhog
Copy Projections structure from EventHandlers
1d91c51
@jakhog
Add state of projections (represented as a JSON string).
071c7ca
@jakhog
Add event key selection specification.
f4a3593
@jakhog
Yes we should have support for scoped projections.
b8e39a2
@woksin
Format and add projection prerelease
7e006f7
@woksin
Don't add to changelog on prereleases
b8a640c
@jakhog
Add initial state to registering, letting head know if projection was…
7830231 
… newly created or already persisted, and handling of delete responses from head.
@jakhog
Move ProjectionEventKeySelector out to own type (looks nicer)
605ff3b
@jakhog
rename ProjectionEventSpecification -> ProjectionEventSelector
62f1149
@jakhog
Fix mistake
7a8e341
@jakhog
We haven't added support for "optional" in the build pipelines yet - …
097d314 
…lets leave that for later. We can send empty strings without much overhead for now (and it would be a non-breaking change in the future).
@woksin
Merge pull request #34 from dolittle/projections
2e610c3 
Adding service definitions for Projections
@jakhog
Update Protobuf package reference, and remove "npx ..." from scripts
5236e9d
@jakhog
Re-set package versions to 0.0.0 (to be set while publishing)
e96693c
@jakhog
Add back JS build and publishing to GitHub workflow
f81cadd
@jakhog
Merge pull request #35 from dolittle/fix-js-contracts-release
d8d242f 
Fix js contracts release
@jakhog
A little quick on the trigger there - fix workflow syntax.
7c01f4d
@jakhog
Only parse semver if we get a version
de7e9b5
@woksin
Merge pull request #36 from dolittle/fix-bug-in-js-release
847bec0 
A little quick on the trigger there - fix workflow syntax.
@jakhog
Split out JS-build + add publish token
5d478c9
@jakhog
Simplify publish checks for JS
f612456
@jakhog
Merge pull request #37 from dolittle/fix-the-fix
ee25684 
Fix the fix
@jakhog
Fix little typo
8eae5e5
@jakhog
Add the "calculated" key to the ProjectionRequest
9e6b037
@jakhog
Merge pull request #38 from dolittle/fix-some-small-things
69f4666 
Fix some small things
@jakhog
Move CSharp projects into its own directory
2fc4c00
@jakhog
Move JavaScript to its own directory
504054f
@jakhog
Fix tsconfig.jsons
bc6fce4
@jakhog
Use right dependencies
67ad292
@jakhog
Fix package.json config
37a5274
jakhog and others added 26 commits March 27, 2021 23:58
@jakhog
Bail out on failure
3d106af
@jakhog
Install protoc
788a999
@jakhog
Install protoc plugins
8258a5b
@jakhog
Test checking out other repository in workflow
8eb9a14
@jakhog
Add Go release job
b6b4ec1
@jakhog
Merge pull request #39 from dolittle/cleanup-generation
cf4a9c3 
Cleanup generation
@jakhog
Add GitHub token for Go.Contracts
5765813
@jakhog
Change workflow to only release code if all builds succeeded
1cd9370
@jakhog
Merge pull request #40 from dolittle/fix-go-release
516dcb7 
Fix go release
@jakhog
Fix JS release
67a641c
@jakhog
Merge pull request #41 from dolittle/fix-js-release
99492b3 
Fix JS release
@jakhog
Delete directories when cleaning out Go code
ef4174f
@jakhog
Merge pull request #42 from dolittle/fix-go-release-to-work-more-than…
ec60036 
…-once

Delete directories when cleaning out Go code
@jakhog
Allow empty commits when releasing Go
66d97e8
@jakhog
Merge pull request #43 from dolittle/allow-unchanged-go-releases
6632b52 
Allow empty commits when releasing Go
@woksin
First iteration for getting projections
f33cf53
@woksin
Add missing import
db48e82
@woksin
GetOne GetAll
f910726
@woksin
Merge pull request #44 from dolittle/get-projection
8f11c80 
First iteration for getting projections
@jakhog
Move projection Key from Request -> CurrentState
d74dea1 
So that the same structure can be reused in the Store.
@jakhog
It has become somewhat of a convention for the failure to go last.
4b5132b
@jakhog
Move the CurrentState over to Projections + rename some files
8800f36
@jakhog
One missing newline forcing me to fix everywhere
2b5b27b
@woksin
Merge pull request #45 from dolittle/add-projection-key-to-states
1a1531a 
Add projection key while getting states + a little cleanup
@jakhog
Changed enums to oneofs
f6295b5
@jakhog
Merge pull request #47 from dolittle/change-enums-to-oneofs
b48d8bb 
Changed enums to oneofs
@woksin woksin added the minor label Apr 9, 2021
@woksin woksin marked this pull request as ready for review April 9, 2021 07:11
@jakhog jakhog merged commit 0a4b556 into master Apr 9, 2021
@jakhog jakhog deleted the 5.3.0-projections branch April 9, 2021 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
minor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jakhog @woksin