Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GitHub Actions file has leftovers from gettsim #134

Closed
janosg opened this issue Mar 27, 2023 · 6 comments · Fixed by #135
Closed

GitHub Actions file has leftovers from gettsim #134

janosg opened this issue Mar 27, 2023 · 6 comments · Fixed by #135

Comments

@janosg
Copy link
Member

janosg commented Mar 27, 2023

This is as snippet from main.yml:

    steps:
      - uses: actions/checkout@v3
      - name: create build environment
        uses: mamba-org/provision-with-micromamba@main
        with:
          environment-file: environment.yml
          environment-name: "{{ cookiecutter.project_slug }}"
          cache-env: true
          extra-specs: |
            python=${{ matrix.python-version }}
      - name: run pytest
        shell: bash -l {0}
        run: |
          micromamba activate gettsim
          pytest --cov-report=xml --cov=./ --ignore=dashboard/
      - name: Upload coverage report.
        if: runner.os == 'Linux' && matrix.python-version == '3.11'
        uses: codecov/codecov-action@v3

It has at least 3 problems:

  1. Once the cookiecutter.project_slug was not replaced
  2. I tries to activate gettsim
  3. The --ignore=dashboard/ is probably a leftover from gettsim
@janosg
Copy link
Member Author

janosg commented Mar 27, 2023

The problems were discovered in this project by @marvinsohn.

hmgaudecker added a commit that referenced this issue Mar 28, 2023
@hmgaudecker hmgaudecker mentioned this issue Mar 28, 2023
@hmgaudecker
Copy link
Member

Thanks! Should be fixed in #135, though I do not understand the first point. Could you elaborate, @janosg @marvinsohn ?

@janosg
Copy link
Member Author

janosg commented Mar 28, 2023

Sure: environment-name: "{{ cookiecutter.project_slug }}" caused github actions to fail because it did not know how to replace cookiecutter.project_slug. I think this should be filled in by cookiecutter, i.e. the generated main.yml should have contained environment-name: option_pricing.

@hmgaudecker
Copy link
Member

Ah, sure, cannot have that as GHA is using the same constructs as cookiecutter.

@janosg
Copy link
Member Author

janosg commented Mar 28, 2023

I think this environment name does not matter at all. Could be a fixed name like testenv if that is easier.

@hmgaudecker
Copy link
Member

Yeah, I fixed it. Needs to be activated below, that's why it matters.

hmgaudecker added a commit that referenced this issue Mar 29, 2023
* Fix paths for LaTeX files (#133).
* Fix GHA workflow (#134).
* Pre-commit autoupdate / fix ruff complaints.
* Use default sequence style of yamlfix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants