Skip to content

Switch back to npm ci#553

Closed
Teja7048 wants to merge 1 commit intoactions:masterfrom
Teja-Nagoori:npm-ci
Closed

Switch back to npm ci#553
Teja7048 wants to merge 1 commit intoactions:masterfrom
Teja-Nagoori:npm-ci

Conversation

@Teja7048
Copy link
Copy Markdown

@Teja7048 Teja7048 commented Jun 20, 2020

Switch back to using npm ci, which is more efficient than npm install

This repository contains configuration for what users see when they click on the Actions tab.

It is not:

  • A playground to try out scripts
  • A place for you to create a workflow for your repository

Thank you for sending in this pull request. Please make sure you take a look at the contributing file. Here's a few things for you to consider in this pull request.

Please add to this description at the bottom 👇

  • A good description of the workflow at the bottom of this page.
  • Some links to the language or tool will be nice (unless its really obvious)

In the workflow and properties files:

  • The workflow filename of CI workflows should be the name of the language or platform, in lower case. Special characters should be removed or replaced with words as appropriate (for example, "dotnet" instead of ".NET").

    The workflow filename of publishing workflows should be the name of the language or platform, in lower case, followed by "-publish".

  • Includes a matching ci/properties/*.properties.json file.

  • Use sentence case for the names of workflows and steps, for example "Run tests".

  • The name of CI workflows should only be the name of the language or platform: for example "Go" (not "Go CI" or "Go Build")

  • Include comments in the workflow for any parts that are not obvious or could use clarification.

  • CI workflows should run on push to branches: [ master ] and pull_request to branches: [ master ].

  • Packaging workflows should run on release with types: [ created ].

Some general notes:

  • This workflow must only use actions that are produced by GitHub, in the actions organization, or
  • This workflow must only use actions that are produced by the language or ecosystem that the workflow supports. These actions must be published to the GitHub Marketplace. Workflows using these actions must reference the action using the full 40 character hash of the action's commit instead of a tag. Additionally, workflows must include the following comment at the top of the workflow file:
    # This workflow uses actions that are not certified by GitHub.
    # They are provided by a third-party and are governed by
    # separate terms of service, privacy policy, and support
    # documentation.
    
  • This workflow must not send data to any 3rd party service except for the purposes of installing dependencies.
  • This workflow must not use a paid service or product.

Switch back to using `npm ci`, which is more efficient than `npm install`
Copy link
Copy Markdown
Author

@Teja7048 Teja7048 left a comment

Choose a reason for hiding this comment

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

Thanks

@ScottBrenner
Copy link
Copy Markdown
Contributor

Merge conflict here

@github-actions
Copy link
Copy Markdown

This pull request has become stale and will be closed automatically within a period of time. Sorry about that.

Copy link
Copy Markdown

@Owenrh0482 Owenrh0482 left a comment

Choose a reason for hiding this comment

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

Commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants