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

don't require a homepage #62

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

Conversation

hellow554
Copy link
Contributor

this is not necessary, neither for cargo nor for bitbake and for our internal projects we typically don't set it, so let's skip it plz :)

this is not necessary, neither for cargo nor for bitbake and for our
internal projects we typically don't set it, so let's skip it plz :)
@kraj
Copy link
Contributor

kraj commented Feb 15, 2023

I think asking for homepage is a good practice. You can keep it empty or undisclosed by saying something there.

@hellow554
Copy link
Contributor Author

you may be right. It's just a thing I always trip over ^^
Because carg-new doesn't fill in neither the repository nor the homepage I have to add it manually.
And just adding an empty string somehow misses the purpose of the field at all, doesn't it?

So as said, for our internal projects, we never add the homepage field, because we know who we are (:
If you still think it's not worth merging, that's fine for me.

@jo-so
Copy link

jo-so commented Nov 27, 2024

I think asking for homepage is a good practice. You can keep it empty or undisclosed by saying something there.

I think cargo-bitbake should not add new policies, but be a helper tool and do (map) what's possible/available.

I use cargo-bitbake as a scaffolder and change the recipes according to our policy (ordering of entries in the recipe). But this check for a homepage blocks it all the time.

@paolobarbolini
Copy link
Contributor

I use cargo-bitbake as a scaffolder and change the recipes according to our policy (ordering of entries in the recipe). But this check for a homepage blocks it all the time.

Have you tried cargo-update-recipe-crates (see #83)?

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.

4 participants