cevi/automatic_walk-time_tables

Remove Duplicate Testing

Closed this issue · 2 comments

wp99cp commented

After merging a PR to the master branch, the following actions run.

image

It would be beneficial to eliminate the testing and development build action at this point. We have no intention of deploying a new version to dev.map.cevi.tools when committing to the master branch, and there is no necessity to test the application again. This is because any commits to the master branch must have passed through a PR, which will always include a testing step.

wp99cp commented

The dev build may be caused by the back merging to dev (I merged the latest version of master back to dev immediately after the original merge commit to master). Maybe here, the GitHub interface is misleading here?

The settings for the dev build are as follows:

on:
push:
branches:
- 'dev'

Anyway, for the testing, we can limit the triggers quite a bit... I propose to enable testing only for active PR and not for every commit in general. @maede97 do you think that is a good idea?

Agree. Let's not waste github's computer power :-)