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

Collections #801

Merged
merged 59 commits into from
Oct 30, 2024
Merged

Collections #801

merged 59 commits into from
Oct 30, 2024

Conversation

josephjclark
Copy link
Collaborator

@josephjclark josephjclark commented Oct 22, 2024 •
edited
Loading

@josephjclark
compiler: support multiple adaptors when handling imports
7a85894
@josephjclark
cli: add support for multiple adaptors on compiler
528e9a0
@josephjclark
compiler: don't extract common exports
8673872
@josephjclark
compiler: be more specific about how we calculate function exports
b6de2c4
@josephjclark
remove the adaptors prop
7245bf7
@josephjclark
cli: working through adaptor -> adaptors changes
ec057fe 
Not done yet
@josephjclark
cli: fix remaining tests after refactor
81cb45c
@josephjclark
worker: refactor to support adaptors array
b15f151
@josephjclark
various: yet another type restructure to handle adaptors safely
a6c3149
@josephjclark
cli:type tweak
1c57b7e
@josephjclark
compiler: update tests
1714d78
@josephjclark
fix type
8305849
@josephjclark
engine: fix test
a71ef46
@josephjclark
tests: fix execute tests
8821bc1
@josephjclark
engine: fix another test
ba42d56
@josephjclark
worker: more typings
ad15608
@josephjclark
Merge pull request #797 from OpenFn/multi-adaptors-compiler-imports
9186fa8 
Support multiple adaptors in the compiler
@josephjclark
tidyup monorepo in preloadAdaptorExports
af97489
@josephjclark
Merge branch 'release/collections' into better-compiler-exports
7fc53b4
@josephjclark
worker: automatically append the collections adaptor to steps that ne…
1c79dc1 
…ed it
@josephjclark
Merge pull request #798 from OpenFn/better-compiler-exports
71edb8b 
compiler: don't extract common exports
@josephjclark
runtime: support global congfiguration on state assembly
3463ff9
@josephjclark
runtime: support global credentials
c6d8ac9
@josephjclark
worker: create global credential for collections
fd0e499
@josephjclark
fixse
4e659a8
@josephjclark
Merge pull request #803 from OpenFn/global-collections
5b2b1f4 
Global credentials
@josephjclark
worker: accept collections version from CLI and refactor some stuff
560fce7
@josephjclark
worker: lookup latest collections version on server start
e9e748b
@josephjclark
worker: changeset
bcd82e9
@josephjclark
worker: update tests
294ca14
@josephjclark
Copy link
Collaborator Author

@stuartc I think this is done.

First job tomorrow is to try it against lightning and see how it holds up.

I don't know if you want to review anything. If you do, check the merged PRs referenced in the description - you'll go insane trying to understand this PR all in one go!

@josephjclark
Copy link
Collaborator Author

Two problems have crept in today:

  1. fn is being imported twice by the compiler. But I fixed this last week!
  2. TypeError: import_undici2.default.client is not a constructor when the mock is disabled

@josephjclark
Copy link
Collaborator Author

Tempted to drop in a fix for #806 here too, because it'll really help testing

This was referenced Oct 29, 2024
Closed
Closed
@josephjclark josephjclark merged commit 451a3fd into main Oct 30, 2024
6 checks passed
@josephjclark josephjclark deleted the release/collections branch October 30, 2024 15:39
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
None yet
Projects
v2
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

integration tests: write a test against collections
1 participant
@josephjclark