Skip to content

Conversation

rose-m
Copy link
Contributor

@rose-m rose-m commented Feb 11, 2021

Use ts-node to directly run the entry script in build/src/index.ts.

@rose-m rose-m self-assigned this Feb 11, 2021
@rose-m rose-m force-pushed the MONGOSH-534-streamline-release-command-execution branch from 6c4cdf7 to 4c0b7ac Compare February 12, 2021 08:47
@rose-m rose-m force-pushed the MONGOSH-534-streamline-release-command-execution branch from 4c0b7ac to b197b8b Compare February 12, 2021 09:53
Copy link
Collaborator

@addaleax addaleax left a comment

Choose a reason for hiding this comment

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

Fwiw, there’s nothing in here that would stick out to me…

@rose-m rose-m force-pushed the MONGOSH-534-streamline-release-command-execution branch from 1e1bae2 to 0f9d889 Compare February 12, 2021 17:12
export NODE_JS_VERSION=${NODE_JS_VERSION}
export ARTIFACT_URL_FILE=artifact-url.txt
export ARTIFACT_URL_FILE="$PWD/artifact-url.txt"
Copy link
Collaborator

Choose a reason for hiding this comment

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

If you want to use absolute paths on Windows, you’ll need to use cygpath -w on the filepath before passing it to Node.js (because this would be /cygdrive/c/.../artifact-url.txt, which on Windows would be interpreted as a path relative to the FS root, e.g. C:\cygdrive\c\...\artifact-url.txt)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I hope the way I've included it now makes that part work...

rose-m and others added 2 commits February 15, 2021 12:23
Co-authored-by: Anna Henningsen <anna.henningsen@mongodb.com>
@rose-m rose-m merged commit af88980 into master Feb 16, 2021
@rose-m rose-m deleted the MONGOSH-534-streamline-release-command-execution branch February 16, 2021 07:36
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