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

fix build errors #157

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

fix build errors #157

wants to merge 1 commit into from

Conversation

janusw
Copy link

@janusw janusw commented Oct 11, 2024

* by removing an extra copy of FreakyZoomableView
@janusw
Copy link
Author

janusw commented Oct 11, 2024

Btw, maybe an automatic CI build would be useful to catch such things? I could contribute some hooks for GitHub Actions, if desired ...

@FreakyAli
Copy link
Owner

@janusw I do have some CI hooks and stuff but they are incomplete, I also need to add tests the unfortunate thing is I have 0 time for the last 3 months

@janusw
Copy link
Author

janusw commented Oct 14, 2024

@janusw I do have some CI hooks and stuff but they are incomplete, I also need to add tests the unfortunate thing is I have 0 time for the last 3 months

Incomplete CI is much better than no CI at all ;)
Why don't you simply activate what you have?

Or are you saying it's already running somewhere (where we don't see it)?

@FreakyAli
Copy link
Owner

FreakyAli commented Oct 14, 2024

I think you should be able to see it, its a github action check this location .github/workflows/main_ci.yml

It's also active as far as I can tell

@janusw
Copy link
Author

janusw commented Oct 14, 2024

I think you should be able to see it, its a github action check this location .github/workflows/main_ci.yml

Right, it's here (thanks for the pointer):

https://github.com/FreakyAli/Maui.FreakyControls/blob/master/.github/workflows/main_ci.yml

It's also active as far as I can tell

Actually I do not see it running anywhere.

First problem is: It's only configured to run on the master branch (upon push or PR), so it's not checking if the develop branch is broken, and it doens't check any PRs to develop.

And even on the master branch, it does not look like it has run for any of the latest commits. On the last commit, it seems like only main_cd.yml has run, and it failed 😢
https://github.com/FreakyAli/Maui.FreakyControls/actions/runs/11193473030/job/31118830311

For #154, it's awaiting your approval, apparently.

@FreakyAli
Copy link
Owner

Like I said it's incomplete and I am not finding much time to complete things!

@janusw
Copy link
Author

janusw commented Oct 15, 2024

I guess the first necessary step would be to merge your master branch into develop, so that the latter also receives the GHA workflows that were added on master. Could you please do that?

Then one can extend those workflows and make sure the build is working (I can help with that) ...

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.

2 participants