Skip to content

Commit a5ca124

Browse files
authored
fix(action): correctly check for package paths (#712)
Co-authored-by: danadajian <danadajian@users.noreply.github.com>
1 parent c3d050a commit a5ca124

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

action/dist/main.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -159784,8 +159784,8 @@ var downloadBaseImages = async () => {
159784159784
await import_fs7.promises.mkdir(screenshotsDirectory, { recursive: true });
159785159785
return;
159786159786
}
159787-
const packagePaths = getInput("package-paths")?.split(",");
159788-
if (packagePaths) {
159787+
const packagePaths = getInput("package-paths")?.split(",").filter(Boolean);
159788+
if (packagePaths?.length) {
159789159789
return Promise.all(
159790159790
packagePaths.map(
159791159791
(packagePath) => downloadS3Directory(
@@ -159805,8 +159805,8 @@ var downloadBaseImages = async () => {
159805159805
var uploadAllImages = async (hash) => {
159806159806
const bucketName = getInput("bucket-name", { required: true });
159807159807
const screenshotsDirectory = getInput("screenshots-directory");
159808-
const packagePaths = getInput("package-paths")?.split(",");
159809-
if (packagePaths) {
159808+
const packagePaths = getInput("package-paths")?.split(",").filter(Boolean);
159809+
if (packagePaths?.length) {
159810159810
return (0, import_bluebird.map)(
159811159811
packagePaths,
159812159812
(packagePath) => uploadLocalDirectory(

action/dist/main.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

action/src/s3-operations.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -184,8 +184,8 @@ export const downloadBaseImages = async () => {
184184
return;
185185
}
186186

187-
const packagePaths = getInput('package-paths')?.split(',');
188-
if (packagePaths) {
187+
const packagePaths = getInput('package-paths')?.split(',').filter(Boolean);
188+
if (packagePaths?.length) {
189189
return Promise.all(
190190
packagePaths.map(packagePath =>
191191
downloadS3Directory(
@@ -207,9 +207,9 @@ export const downloadBaseImages = async () => {
207207
export const uploadAllImages = async (hash: string) => {
208208
const bucketName = getInput('bucket-name', { required: true });
209209
const screenshotsDirectory = getInput('screenshots-directory');
210-
const packagePaths = getInput('package-paths')?.split(',');
210+
const packagePaths = getInput('package-paths')?.split(',').filter(Boolean);
211211

212-
if (packagePaths) {
212+
if (packagePaths?.length) {
213213
return map(packagePaths, packagePath =>
214214
uploadLocalDirectory(
215215
path.join(screenshotsDirectory, packagePath),

0 commit comments

Comments
 (0)