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

fix reference to cql path #3

Merged
merged 1 commit into from
Jul 12, 2022
Merged

Conversation

achen2401
Copy link
Contributor

remove relative path when importing cql-worker module

Copy link
Member

@ivan-c ivan-c left a comment

Choose a reason for hiding this comment

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

Tested locally, works great. Thanks much!!!

@ivan-c ivan-c mentioned this pull request Jul 12, 2022
@achen2401 achen2401 merged commit 5efc423 into main Jul 12, 2022
@achen2401 achen2401 deleted the bugfix/cql-worker-reference-path branch July 12, 2022 22:23
ivan-c added a commit that referenced this pull request Jul 12, 2022
ivan-c added a commit that referenced this pull request Jul 20, 2022
* Add initial docker files

* WIP add dev-related docker files

* WIP try installing node_modules into parent directory of checkout

* WIP

* Run as non-root inside container; fix paths

* Cleanup comment; add missing chown

* fixup comment

* reword

* Add comment

* Use long options

* Rearrange dockerfile lines

* Add docs, example config

* Fixup markdown

* Move USER further down

* Run dc in foreground

* Remove workaround; fixed in #3

* Refactor environment files

* Fixup spelling

Co-authored-by: Hannah <ha.al.bu@gmail.com>

Co-authored-by: Hannah <ha.al.bu@gmail.com>
hannahburkhardt referenced this pull request in uwcirg/isacc-messaging-client-sof Aug 11, 2022
* Add initial docker files

* WIP add dev-related docker files

* WIP try installing node_modules into parent directory of checkout

* WIP

* Run as non-root inside container; fix paths

* Cleanup comment; add missing chown

* fixup comment

* reword

* Add comment

* Use long options

* Rearrange dockerfile lines

* Add docs, example config

* Fixup markdown

* Move USER further down

* Run dc in foreground

* Remove workaround; fixed in #3

* Refactor environment files

* Fixup spelling

Co-authored-by: Hannah <ha.al.bu@gmail.com>

Co-authored-by: Hannah <ha.al.bu@gmail.com>
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