Skip to content

fix: suppress manifest.json output in action logs - #121

Open
SuperPiIot wants to merge 1 commit into
deployphp:masterfrom
SuperPiIot:fix-issue-88
Open

SuperPiIot wants to merge 1 commit into
deployphp:masterfrom
SuperPiIot:fix-issue-88

Conversation

@SuperPiIot

Copy link
Copy Markdown

Summary

Temporarily disable $.verbose while fetching https://deployer.org/manifest.json so the full manifest payload is not dumped into the GitHub Actions log when resolving the Deployer version.

Closes #88
/claim #88

@alexisLefebvre

Copy link
Copy Markdown

The other PR didn’t give the expected result?

@SuperPiIot

Copy link
Copy Markdown
Author

Hi @alexisLefebvre!

PR #89 targeted the older index.js file before the repository was migrated to TypeScript. Since the current build pipeline compiles from src/index.ts, PR #121 brings the fix directly to the active TypeScript source.

Both approaches solve the issue, but this PR silences the curl output in the current build without changing the underlying transport. If you prefer switching to fetch() in src/index.ts instead of $.verbose = false, I'd be happy to update it accordingly!

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.

How to remove manisfest from log ?

2 participants

⌂ Proxy