Skip to content

Add GNUtar as default in windows - #1232

Merged
Phantsure merged 14 commits into
mainfrom
phantsure/gnutar_windows
Nov 16, 2022
Merged

Add GNUtar as default in windows#1232
Phantsure merged 14 commits into
mainfrom
phantsure/gnutar_windows

Conversation

@Phantsure

Copy link
Copy Markdown
Contributor

GNUtar is present in windows hosted runners to default to it for archiving

@Phantsure
Phantsure requested a review from lvpx November 7, 2022 13:47
@Phantsure
Phantsure force-pushed the phantsure/gnutar_windows branch from ff3291a to 436cf8d Compare November 10, 2022 15:52
@Phantsure
Phantsure marked this pull request as ready for review November 14, 2022 05:25
@Phantsure
Phantsure requested a review from a team as a code owner November 14, 2022 05:25
Comment thread packages/cache/src/internal/cacheUtils.ts Outdated
Comment thread packages/cache/src/internal/cacheHttpClient.ts Outdated

@lvpx lvpx left a comment

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.

Some minor comments.

Comment thread packages/cache/__tests__/tar.test.ts Outdated
Comment thread packages/cache/__tests__/tar.test.ts
Comment thread packages/cache/src/internal/tar.ts Outdated
Comment thread packages/cache/src/internal/tar.ts Outdated
Comment thread packages/cache/src/internal/tar.ts

@lvpx lvpx left a comment

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.

LGTM. A minor comment related to imports. Please take care of failing test cases other than the audit one.

@Phantsure

Copy link
Copy Markdown
Contributor Author

Done

Comment thread packages/cache/src/internal/cacheUtils.ts Outdated
Comment thread packages/cache/src/internal/cacheUtils.ts Outdated
Comment thread packages/cache/src/internal/tar.ts Outdated
Comment thread packages/cache/src/internal/tar.ts Outdated
Comment thread packages/cache/src/internal/cacheUtils.ts
Comment thread packages/cache/src/internal/constants.ts Outdated
Comment thread packages/cache/src/internal/cacheUtils.ts
Comment thread packages/cache/src/internal/tar.ts Outdated
Comment thread packages/cache/src/internal/tar.ts Outdated
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.

3 participants