macports / upt-macports

The Universal Packaging Tool (upt) backend for MacPorts
https://framagit.org/upt/upt
BSD 3-Clause "New" or "Revised" License
8 stars 12 forks source link

Add python name test #24

Closed Korusuke closed 5 years ago

Korusuke commented 5 years ago

Once this and the other tests of perl and ruby are merged the coverage should reach about 85%.

reneeotten commented 5 years ago

let's first make sure this is indeed the naming convention we decide on before adding tests ;) I have just commented in the Issue. Let's see if Josh or Ryan respond (they will know if there are implicit naming conventions) and take it from there.

reneeotten commented 5 years ago

okay, moving forward we'll use the naming py-<upstream> including capitalization; that certainly makes life easier. We might need to deal with some flexibility for the time being, but let's get to that when working later on checking for//recursive addition of dependencies...

Please rephrase the commit message so that it's clear that we use "py-" as MacPorts package name (the tests are implied) and then I will merge.

Korusuke commented 5 years ago

Should we just revert this merge and do the changes here?

reneeotten commented 5 years ago

Should we just revert this merge and do the changes here?

If it's up to me: no - I am not in favor of reverting already merged PRs.