OpenEnergyPlatform / open-MaStR

A collaborative software to download the energy database Marktstammdatenregister (MaStR)
https://open-mastr.readthedocs.io/en/latest/
GNU Affero General Public License v3.0
87 stars 19 forks source link

Change while loop to for loop #450

Closed jessymatar closed 1 year ago

jessymatar commented 1 year ago

Summary of the discussion

With a for loop, we could run a specific number of iterations unlike while. Also it is a test run to get to know the techniques with pull request.

Type of change (CHANGELOG.md)

Updated

Workflow checklist

PR-Assignee

Reviewer

FlorianK13 commented 1 year ago

@chrwm Do you understand why those tests are failing?

FlorianK13 commented 1 year ago

@chrwm @jessymatar I finally figured out why those tests are failing: Github Secrets are not passed to Workflows from Pull Requests coming from forks (for security reasons). Hence these tests cannot use the API credentials and hence the tests fail. How should we proceed here @chrwm ?

FlorianK13 commented 1 year ago

This is also implemented in #470