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.

Sign up for GitHub

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

Jump to bottom

Update build-publish-linux.yml to use autdiwheel #60

Closed
wants to merge 2 commits into from
Closed

Update build-publish-linux.yml to use autdiwheel #60

wants to merge 2 commits into from

Conversation

MatPoliquin
Copy link
Collaborator

Attempt to use auditwheel to make the wheel compliant with manylinux

@pseudo-rnd-thoughts
Copy link
Member

Could we fix the link checker or remove it

@MatPoliquin
Copy link
Collaborator Author

Could we fix the link checker or remove it

I would be in favor of removing it for now as it's not very reliable, sometimes it works sometimes it doesn't.
Maybe it's a problem with our integration but there is more urgent priorities for now.
I would add the task to Monday.com board but still don't have permissions

@zbeucler2018 zbeucler2018 mentioned this pull request Aug 25, 2023
Merged
@zbeucler2018
Copy link
Collaborator

Whats the status on this? I'd like to help if needed

@pseudo-rnd-thoughts
Copy link
Member

@zbeucler2018 could you make a Pr to remove the link checker?

@zbeucler2018
Copy link
Collaborator

@zbeucler2018 could you make a Pr to remove the link checker?

Already done on MR#63 !

@MatPoliquin MatPoliquin deleted the MatPoliquin-patch-1 branch September 25, 2023 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MatPoliquin @pseudo-rnd-thoughts @zbeucler2018