actions/languageservices
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove support for */TUE

Browse Source
This commit is contained in:
Jacob Wallraff
2023-01-10 11:54:41 -08:00
parent 92db0c1af7
commit f877c22e9c
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
actions-workflow-parser/src/model/converter/cron.test.ts
+2 -1
View File
@@ -18,7 +18,6 @@ describe("isValidCron", () => {
"0 0 * * SUN",
"0 0 * * SUN-TUE",
"0 0 * * SUN-2",
"0 * * */FEB */TUE",
"0 2-4/5 * * *",
]
@@ -40,6 +39,7 @@ describe("isValidCron", () => {
"0 5--5 * * *",
"0 *//5 * * *",
"0 ,, * * *",
"0 , , , ,",
"0 ** * * *",
"0 0 * * BUN",
"0 0 * SUN JAN",
@@ -49,6 +49,7 @@ describe("isValidCron", () => {
"0 2/4-5 * * *",
"0 2-4-6 * * *",
"0 2/4/6 * * *",
"0 * * */FEB */TUE",
]
for (const cron of invalid) {
actions-workflow-parser/src/model/converter/cron.ts
+2 -2
View File
@@ -76,8 +76,8 @@ function validateRange(
}
const [start, step, ...rest] = value.split("/")
const stepNumber = convertToNumber(range, step)
if (rest.length > 0 || stepNumber <= 0 || !start || !step) {
const stepNumber = +step
if (rest.length > 0 || isNaN(stepNumber) || stepNumber <= 0 || !start || !step) {
return false
}