Skip to content

fix the publisher CI#140

Merged
EliMoshkovich merged 1 commit intomainfrom
PER-13365-fix-publisher
Nov 10, 2025
Merged

fix the publisher CI#140
EliMoshkovich merged 1 commit intomainfrom
PER-13365-fix-publisher

Conversation

@EliMoshkovich
Copy link
Contributor

Pull Request

Description

Type of Change

  • Bug fix
  • New feature/command
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test addition/update
  • Other (please describe):

Checklist

  • I have created an issue and linked it in this PR
  • I have created a branch from main with an appropriate name (e.g., fix/issue-123, feature/new-command)
  • My code follows the project's coding style guidelines
  • I have added tests for my changes (>90% coverage of new code)
  • I have updated the documentation if necessary
  • All tests pass locally
  • Lint checks pass locally
  • I have reviewed my own code for potential issues

New Command Details (if applicable)

  • Command is placed in the src/commands directory
  • Command file contains only argument configuration and a root command component
  • Command is wrapped with the AuthProvider component
  • Command has an optional apiKey argument
  • API key scope is declared for the command
  • Documentation added to the README

Additional Notes

Screenshots/Recordings

@linear
Copy link

linear bot commented Nov 10, 2025

@EliMoshkovich EliMoshkovich merged commit f614989 into main Nov 10, 2025
3 checks passed
@EliMoshkovich EliMoshkovich deleted the PER-13365-fix-publisher branch November 10, 2025 15:09
Tabintel pushed a commit to Tabintel/permit-cli that referenced this pull request Dec 5, 2025
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.

3 participants