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

cosmosdb-dmt 2.1.4 (new cask) #168865

Closed
wants to merge 2 commits into from
Closed

Conversation

crpietschmann
Copy link

@crpietschmann crpietschmann commented Mar 12, 2024

The Azure Cosmos DB Desktop Data Migration Tool is an open-source project containing a command-line application that provides import and export functionality for Azure Cosmos DB.

Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

In the following questions <cask> is the token of the cask you're submitting.

After making any changes to a cask, existing or new, verify:

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • Checked the cask was not already refused.
  • Checked the cask is submitted to the correct repo.
  • brew audit --cask --new <cask> worked successfully.
  • HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask> worked successfully.
  • brew uninstall --cask <cask> worked successfully.

The Azure Cosmos DB Desktop Data Migration Tool is an open-source project containing a command-line application that provides import and export functionality for Azure Cosmos DB.
@BrewTestBot BrewTestBot added new cask missing description Cask is missing a description, please add one. missing zap Cask is missing a zap stanza, please add one. labels Mar 12, 2024
@crpietschmann
Copy link
Author

FYI, I am a Developer on the Azure Cosmos DB Data Migration Tool project (https://github.com/AzureCosmosDB/data-migration-desktop-tool). I may need to follow up with getting the binary signed with Apple for macOS installation, if there's no workaround to get this cask committed to Homebrew.

Is there a flag to get the Cask submitted without the Apple binary signing?

@krehel
Copy link
Member

krehel commented Mar 12, 2024

FYI, I am a Developer on the Azure Cosmos DB Data Migration Tool project (AzureCosmosDB/data-migration-desktop-tool). I may need to follow up with getting the binary signed with Apple for macOS installation, if there's no workaround to get this cask committed to Homebrew.

Is there a flag to get the Cask submitted without the Apple binary signing?

We cannot accept it at this time because it does not pass our signature verification check that is a requirement for new submissions. The binary should be signed and notarized for acceptance.

@krehel krehel added the awaiting user reply Issue needs response from a user. label Mar 12, 2024
Updated indentation and shortened description
@BrewTestBot BrewTestBot removed the missing description Cask is missing a description, please add one. label Mar 12, 2024
@crpietschmann
Copy link
Author

FYI, I am a Developer on the Azure Cosmos DB Data Migration Tool project (AzureCosmosDB/data-migration-desktop-tool). I may need to follow up with getting the binary signed with Apple for macOS installation, if there's no workaround to get this cask committed to Homebrew.
Is there a flag to get the Cask submitted without the Apple binary signing?

We cannot accept it at this time because it does not pass our signature verification check that is a requirement for new submissions. The binary should be signed and notarized for acceptance.

Thanks for clarification. I'm going to close this PR for now then. We'll have to circle back to this accordingly in the future.

@krehel
Copy link
Member

krehel commented Mar 12, 2024

@crpietschmann - can this be built entirely from source and is a CLI app? If so it probably would be better as a Formula in homebrew-core instead of a Cask.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting user reply Issue needs response from a user. missing zap Cask is missing a zap stanza, please add one. new cask
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants