Skip to content

Fix: stdout match #38

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
Merged
merged 1 commit into from
Jul 5, 2021
Merged

Fix: stdout match #38

merged 1 commit into from
Jul 5, 2021

Conversation

@luhc228
Copy link
Collaborator

@luhc228 luhc228 commented Jul 5, 2021 •
edited
Loading

stdout 一般是下面的内容。按照旧的正则表达式，无法匹配上正确的结果

=> Downloading nvm from git to '/Users/luhc228/.nvm'\n\r=> Cloning into '/Users/luhc228/.nvm'...\n* (HEAD detached at FETCH_HEAD)\n  master\n=> Compressing and cleaning up git repository\n\n=> Appending nvm source string to /Users/luhc228/.profile\n=> bash_completion source string already in /Users/luhc228/.profile\nDebugger attached.\nWaiting for the debugger to disconnect...\nDebugger attached.\nWaiting for the debugger to disconnect...\n=> The Local Node is not found. It will install node v14.17.2 versio…############################### 100.0%\nComputing checksum with shasum -a 256\nChecksums matched!\nnvm is not compatible with the "npm_config_prefix" environment variable: currently set to "/Users/luhc228/.volta/tools/image/node/14.17.0"\nRun `unset npm_config_prefix` to unset it.\nFailed to install Node.js v14.17.2\n=> Close and reopen your terminal to start using nvm or run the following to use it now:\n\nexport NVM_DIR="$HOME/.nvm"\n[ -s "$NVM_DIR/nvm.sh" ] && \\. "$NVM_DIR/nvm.sh"  # This loads nvm\n

@luhc228 luhc228 requested a review from fyangstudio July 5, 2021 06:03
@luhc228 luhc228 self-assigned this Jul 5, 2021
@luhc228 luhc228 merged commit 4104f33 into release-next Jul 5, 2021
@luhc228 luhc228 deleted the fix/stdout-match branch July 5, 2021 06:09
luhc228 added a commit that referenced this pull request Jul 6, 2021
@luhc228
Release 0.1.1 (#24)
c5d93d8 
* chore: version

* fix: icon size

* fix: icon file size

* Feat: record DAU (#25)

* fix: stdout match (#38)

* fix: fail to install nvm (#29)

* Fix: fail to install code to path (#37)

* Feat: auto download package (#31)

* Fix: vscode cli command not found (#36)

* fix: nvm has installed but actually not installed (#39)

* fix: managerVersion not found (#40)

* Fix: electron not found in prod env (#41)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fyangstudio fyangstudio fyangstudio approved these changes

Assignees

@luhc228 luhc228

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@luhc228 @fyangstudio