Skip to content

bug fix upload artifact#127

Open
nikhil-mittal-165 wants to merge 2 commits intomainfrom
feature/upload-artifact-bug-fix
Open

bug fix upload artifact#127
nikhil-mittal-165 wants to merge 2 commits intomainfrom
feature/upload-artifact-bug-fix

Conversation

@nikhil-mittal-165
Copy link
Copy Markdown
Contributor

No description provided.

@git2gus
Copy link
Copy Markdown

git2gus Bot commented May 4, 2026

Git2Gus App is installed but the .git2gus/config.json doesn't have right values. You should add the required configuration.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Salesforce Code Analyzer found 13 violations, including 8 in files changed by this pull request. See job summary page.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Salesforce Code Analyzer found 13 violations, including 8 in files changed by this pull request. See job summary page.

@nikhil-mittal-165
Copy link
Copy Markdown
Contributor Author

Manually threw an exception of GHES to replicate the issue

Current Scenario

manually throwing exception to replicate GHES error

image

what happens internally other jobs get cancelled

image

Fixed Scenario
fixed code where we did graceful exception handling

image

I see a warning other actions succeded

image

@nikhil-mittal-165 nikhil-mittal-165 force-pushed the feature/upload-artifact-bug-fix branch from 0ac5f1b to 55fdffa Compare May 6, 2026 09:58
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Salesforce Code Analyzer found 13 violations, including 8 in files changed by this pull request. See job summary page.

Comment thread dist/index.js
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why do we need to push dist folder ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

the code build failed if i was not pushing it, seems for github action the index file should be up to date as they dont run npm install while running the action

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

check this PR as well : https://github.com/forcedotcom/run-code-analyzer/pull/77/changes

here it has pushed index.js file

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