chore: Remove npm i stage from actino, allowing users to use whatever package manager they want i.e. yarn

Author: thelukewalton

README.md

@@ -43,6 +43,9 @@ jobs:
           node-version-file: .node-version
           cache: npm
 
+      - name: Install dependencies
+        run: npm ci
+
       - name: Code Quality
         uses: ZebraDevs/web-components-code-quality@v1
         with:

package-lock.json

@@ -283,14 +283,13 @@ const getInputs = (
  * 1. Check if the environment is local.
  * 2. Retrieve inputs based on the environment.
  * 3. Change the working directory if specified.
- * 4. Install dependencies using npm.
- * 5. Generate a Custom Elements Manifest using `npx cem analyze`.
- * 6. Run static analysis tools (ESLint, Lit Analyzer, TypeDoc) if enabled.
- * 7. Format code using Prettier if enabled.
- * 8. Install Playwright browsers and run tests if enabled.
- * 9. Calculate and compare code coverage if enabled.
- * 10. Check for modified files and update changes in the GitHub repository if any.
- * 11. Create a comment on the GitHub pull request with the results if enabled.
+ * 4. Generate a Custom Elements Manifest using `npx cem analyze`.
+ * 5. Run static analysis tools (ESLint, Lit Analyzer, TypeDoc) if enabled.
+ * 6. Format code using Prettier if enabled.
+ * 7. Install Playwright browsers and run tests if enabled.
+ * 8. Calculate and compare code coverage if enabled.
+ * 9. Check for modified files and update changes in the GitHub repository if any.
+ * 10. Create a comment on the GitHub pull request with the results if enabled.
  */
 export async function run(): Promise<void> {
   const isLocal = checkIfLocal();
@@ -317,12 +316,6 @@ export async function run(): Promise<void> {
       chdir(workingDirectory);
     }
 
-    // run set up
-    const npmIStr: StepResponse | undefined = await commandComment({
-      label: "Install Dependencies",
-      command: "npm i --ignore-scripts",
-    });
-
     const cemStr: StepResponse | undefined = await commandComment({
       label: "Custom Elements Manifest",
       command: "npx cem analyze",
@@ -429,7 +422,6 @@ export async function run(): Promise<void> {
       await comment(
         getOctokit(token),
         context,
-        npmIStr,
         cemStr,
         eslintStr,
         litAnalyzerStr,

src/main.ts

@@ -48,7 +48,6 @@ const li = (str: string): string => {
  *
  * @param {ReturnType<typeof getOctokit>} ocotokit - The Octokit instance for making GitHub API requests.
  * @param {Context} context - The context of the GitHub action, including issue and repository information.
- * @param {StepResponse | undefined} npmIStr - The result of the npm install step.
  * @param {StepResponse | undefined} cemStr - The result of the custom element manifest step.
  * @param {StepResponse | undefined} eslintStr - The result of the ESLint step.
  * @param {StepResponse | undefined} litAnalyzerStr - The result of the Lit Analyzer step.
@@ -64,7 +63,6 @@ const li = (str: string): string => {
 export const comment = async (
   ocotokit: ReturnType<typeof getOctokit>,
   context: Context,
-  npmIStr: StepResponse | undefined,
   cemStr: StepResponse | undefined,
   eslintStr: StepResponse | undefined,
   litAnalyzerStr: StepResponse | undefined,
@@ -82,7 +80,6 @@ export const comment = async (
     let formatting = [];
     let testing = [];
     let postChecks = [];
-    if (npmIStr !== undefined) setup.push(npmIStr);
     if (cemStr !== undefined) setup.push(cemStr);
     if (eslintStr !== undefined) analysis.push(eslintStr);
     if (litAnalyzerStr !== undefined) analysis.push(litAnalyzerStr);
@@ -108,7 +105,7 @@ ${group("Post Checks", postChecks, false)}
       owner: context.repo.owner,
       repo: context.repo.repo,
     });
-    const comment = comments.find((comment) =>
+    const comment = comments.find((comment: any) =>
       // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
       comment.body!.includes("PR Checks Complete"),
     );