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.

Sign up for GitHub

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

Jump to bottom

asyncapi 0.35.0 #127077

Closed
wants to merge 1 commit into from
Closed

asyncapi 0.35.0 #127077

wants to merge 1 commit into from

Conversation

asyncapi-bot-eve
Copy link
Contributor

@BrewTestBot BrewTestBot added the nodejs Node or npm use is a significant feature of the PR or issue label Mar 30, 2023
@carlocab
Copy link
Member

Hi, @derberg. I noticed you're the one who added the workflow for this automated version bump.

We've had 3 PRs today from @asyncapi-bot-eve. This isn't the first time we've had multiple PRs in a day from your bot. Our CI is self-hosted (and limited!), so we'd appreciate it if there were some way to limit the number of PRs opened on our side.

@carlocab carlocab force-pushed the update-asyncapi.rb-1680185673 branch from d8efd4c to abbfc9f Compare March 30, 2023 15:41
@p-linnane p-linnane closed this Mar 30, 2023
@p-linnane p-linnane added the superseded PR was replaced by another PR label Mar 30, 2023
@chenrui333
Copy link
Member

#127106

@chenrui333
Copy link
Member

cc @derberg

@derberg
Copy link
Contributor

derberg commented Apr 3, 2023

@carlocab @chenrui333 oh crap. So sorry folks. I really did not want to cause any issues in your pipelines. Sorry for being an ass. It happened this time and before as this is how I maintain our CLI, just merge bunch of stuff every couple of days when I have a moment. I'll make sure to stop it and inform other maintainers of the CLI. Long term we will probably have to switch to one daily PR to brew in case there was a release of the CLI, to batch few minors in one PR.

sorry sorry again and thanks to all of your support 🙏🏼

@derberg derberg mentioned this pull request Apr 3, 2023
Closed
@chenrui333
Copy link
Member

@derberg no worries, and appreciate the fix on the pipeline improvements! 🍻

@carlocab
Copy link
Member

carlocab commented Apr 4, 2023

Thanks @derberg!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
nodejs Node or npm use is a significant feature of the PR or issue superseded PR was replaced by another PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@asyncapi-bot-eve @carlocab @chenrui333 @derberg @BrewTestBot @p-linnane