asdf-r icon indicating copy to clipboard operation
asdf-r copied to clipboard

Updates the README to resolve #10

Open signus opened this issue 2 years ago • 2 comments

Based on #10 the issue is itself not actually with the install but the instructions provided to install the ASDF plugin for R.

This is just a single line change in the README to resolve the message R is invalid. Name may only contain lowercase letters, numbers, '_', and '-':

asdf plugin-add r https://github.com/asdf-community/asdf-r.git

signus avatar May 09 '23 16:05 signus

Note that the hygiene check seems to be based on an outdated workflow that does not match with the current Makefile:

asdf-community/.github/.github/required-workflows/hygiene.yml

name: Hygiene

on:
  - pull_request

jobs:
  hygiene:
    name: Hygiene

    runs-on: ubuntu-latest

    steps:
      - name: Checkout code
        uses: actions/checkout@v3

      - name: Set-up Homebrew
        uses: Homebrew/actions/setup-homebrew@master

      # Dependencies
      - run: brew install shellcheck shfmt

      # Check formatting with shfmt
      - run: make format-check

      # Check linting with shellcheck
      - run: make lint

signus avatar May 09 '23 17:05 signus

You need to also fix the example lines here: https://github.com/asdf-community/asdf-r/blob/f1e4b8b31bcbdbe36c852bd3696dfcdc247ea883/README.md#L40-L44

Should be asdf install r, not R

flying-sheep avatar Aug 22 '23 11:08 flying-sheep