ittybittyapps / appstoreconnect-cli

An easy to use command-line tool for interacting with the Apple AppStore Connect API
MIT License
172 stars 17 forks source link

Upgrade Yams dependency to 5.0.1 #252

Closed orj closed 2 years ago

orj commented 2 years ago

📝 Summary of Changes

Changes proposed in this pull request:

⚠️ Items of Note

🧐🗒 Reviewer Notes

🔨 How To Test

Test asc commands that load config from Yaml files.

JamesW-iOS commented 2 years ago

Could be possible I'm just using it wrong but it's erroring out when I try to sync user information from a YAML file.

Command:

asc users sync ./input.yml --input-format yaml --dry-run

input.yml:

- username: me@thetrex.dev
  firstName: James
  lastName: TestLastName
  roles:
  - ADMIN
  - ACCOUNT_HOLDER
  provisioningAllowed: true
  allAppsVisible: true

Output:

## Dry run ##
Error: Expected single publisher output but received 0
orj commented 2 years ago

@JamesW-iOS Could also be that it doesn't work. :)

orj commented 2 years ago

@JamesW-iOS This bug isn't an issue with the Yaml library but rather with the rest of the code.