Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

Feat/advanced publish #241

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Feat/advanced publish #241

wants to merge 7 commits into from

Conversation

Jakub-Vacek
Copy link
Member

@Jakub-Vacek Jakub-Vacek commented Nov 18, 2022

Description

This PR improves Publish command using Service Client in dry run mode to catch errors before actual publish. This also enables to publish multiple file types (profile, map, provider) in one run. Missing tests, clean up and maybe some command interface update (interactive?). Also should be able to prefix local provider to unverified- before publishing,

Motivation and Context

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have updated the documentation accordingly. For updating Oclif commands documentation use oclif-dev.
  • I have read the CONTRIBUTION_GUIDE document.
  • I haven't repeated the code. (DRY)
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@freaz freaz changed the base branch from dev to main February 8, 2023 15:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant