Skip to content

Conversation

@poojadonode28
Copy link
Collaborator

No description provided.

@poojadonode28 poojadonode28 force-pushed the RTDEV-57711-archive-old-artifacts-worker-sample-for-deprecated-plugin branch 5 times, most recently from 9a2e130 to 087c733 Compare May 29, 2025 05:29
@poojadonode28 poojadonode28 force-pushed the RTDEV-57711-archive-old-artifacts-worker-sample-for-deprecated-plugin branch from 087c733 to 47ba554 Compare May 29, 2025 09:16
Copy link
Contributor

Choose a reason for hiding this comment

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

It could be worth adding some Markdown formatting to make this README more readable.
Like for instance this README file.

Copy link
Contributor

Choose a reason for hiding this comment

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

This Worker takes a lot of parameters.
Maybe adding few usage examples could make it clearer ?

}

} catch (Error) {
console.log("Artifactory archieved failed");
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
console.log("Artifactory archieved failed");
console.error("Artifactory archieved failed: " + Error.message);

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