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

Merge extractpo & extractpo-oc into a single one #65

Merged
merged 2 commits into from
Dec 3, 2022
Merged

Conversation

hishamco
Copy link
Member

Fixes #64

@hishamco
Copy link
Member Author

@lukaskabrt why there's a red mesage when I created a PR, is this related to some permissions?

@lukaskabrt
Copy link
Collaborator

@hishamco I don't see any red message, but it seems there is some problem with PR checks.

It might be possible that moving the project under OrchardCoreContrib broke the integration with Azure DevOps, because I don't see any new builds there.

I don't see project settings here at GitHub anymore, so I am unable to check it further. Can you access DevOps project at https://dev.azure.com/kabrt/PoExtractor ? I have added you as a project contributor.

@hishamco
Copy link
Member Author

It might be possible that moving the project under OrchardCoreContrib broke the integration with Azure DevOps, because I don't see any new builds there.

I removed teh AzurePipelines checks and using GitHub workflow

Merge without waiting for requirements to be met (bypass branch protections)

This is the message show up above the merge button, I will check the setting again, otherwise I will give you an access to check if you have time

Thanks

@hishamco
Copy link
Member Author

There was a rule on the master branch, I delete it and everything is fine now ;)

@hishamco hishamco merged commit 1843068 into master Dec 3, 2022
@hishamco hishamco deleted the poextract-tool branch December 3, 2022 14:29
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.

Merge PoExtractor & PoExtractor.OrchardCore into a single tool
2 participants