Run builds on every PR change

[tovbinm]

Description

This to allow enabling branch protection. Since we require a Test to pass prior to merging for all contributions (including administrators), this change is necessary.

[thomas-gerber]

Description

This to allow enabling branch protection

I do not think we need to run build on every PR to be comfortable with branch protection for admins. Indeed, if we are in a bind, an admin can always go to settings, allows admins to bypass branch protection, fix things.

That being said, running build on every PR may be a good idea in its own right. I am in favor of it.

[thomas-gerber]

Description

This to allow enabling branch protection

I do not think we need to run build on every PR to be comfortable with branch protection for admins. Indeed, if we are in a bind, an admin can always go to settings, allows admins to bypass branch protection, fix things.

That being said, running build on every PR may be a good idea in its own right. I am in favor of it.

I misunderstood the description. This is only to run the existing PR workflow even if we only modify readme files, to allow merging PRs because of the check.

[tovbinm]

Since we require a Test to pass prior to merging for all contributions (including administrators), this change is necessary.