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

Added property type support to Drupal.Commenting.VariableComment #63

Closed
wants to merge 3 commits into from

Conversation

arkener
Copy link
Collaborator

@arkener arkener commented Nov 22, 2019

@klausi
Copy link
Collaborator

klausi commented Nov 30, 2019

Hm, why are Travis CI tests not executing here? Can you merge in the 8.x-3.x branch and see if that triggers the tests?

@arkener
Copy link
Collaborator Author

arkener commented Nov 30, 2019

Merged the 8.x-3.x branch from upstream, however the CI tests still don't seem to be triggered. Do you have any idea why this would be the case?

@jonathan1055
Copy link
Contributor

On https://travis-ci.org/pfrenssen/coder/pull_requests if you select the righthand 'more options' and pick 'requests' it shows the failed ones as well. It say "Abuse detected" for these, so maybe your IP is on a banned list or something?

@jonathan1055
Copy link
Contributor

jonathan1055 commented Nov 30, 2019

The previous non-run was 8 days ago which also matches your previous try.

@arkener
Copy link
Collaborator Author

arkener commented Nov 30, 2019

My account seems to be flagged on Travis CI. I've contacted support to figure out why this is the case.

@klausi
Copy link
Collaborator

klausi commented Nov 30, 2019

created #67 to trigger Travis CI in the meantime

@klausi
Copy link
Collaborator

klausi commented Nov 30, 2019

manually merged. thanks!

@klausi klausi closed this Nov 30, 2019
@arkener arkener deleted the typedproperty branch December 3, 2019 13:33
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 this pull request may close these issues.

None yet

3 participants