Remove explicit pull request event types from workflow

[rdmark]

According to github docs, when not specifying pull request event type in the github workflow, the default events are "opened, synchronize, and reopened". However, the way we explicitly defined the event types caused the "synchronize" event (run the jobs when new commit is synchronized with the remote branch) to not trigger. Removing the explicitly defined events and using the default gets us the wanted behavior (e.g. triggering workflows on new commits to a PR branch.)

[rdmark]

We seem to have been using the wrong syntax for events. The documentation suggests the array syntax for the list of event types. I tested this syntax:

  pull_request:
    paths:
      - 'cpp/**'
      - 'python/**'
      - '.github/workflows/cpp.yml'
    types: [assigned, opened, reopened, synchronize]

However this resulted in the duplicate sets of jobs, again. I'm not sure if this is expected, or a bug in github workflows.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[uweseimet]

@rdmark Thank you for analyzing this.

[rdmark]

@uweseimet You're welcome. Let me know if this works correctly for your workflow as well. I tested that subsequent changes to C++ or Python code triggered the jobs that I expected.

[uweseimet]

@rdmark I'm afraid there is no change. I just committed d1a080b and the anaylsis was not started.