Skip to content

fix: typo unmarshalling#2069

Closed
manuelarte wants to merge 6 commits into
oapi-codegen:mainfrom
manuelarte:fix/unmarshalling-typo
Closed

fix: typo unmarshalling#2069
manuelarte wants to merge 6 commits into
oapi-codegen:mainfrom
manuelarte:fix/unmarshalling-typo

Conversation

@manuelarte

@manuelarte manuelarte commented Aug 29, 2025

Copy link
Copy Markdown

I found some typos:

  • unmarshaling -> unmarshalling.
  • specificiation -> specification

@manuelarte manuelarte requested a review from a team as a code owner August 29, 2025 13:34
@kusari-inspector

kusari-inspector Bot commented Aug 29, 2025

Copy link
Copy Markdown

Kusari Inspector

Kusari Analysis Results:

Proceed with these changes

✅ No Flagged Issues Detected
All values appear to be within acceptable risk parameters.

No pinned version dependency changes, code issues or exposed secrets detected!

Note

View full detailed analysis result for more information on the output and the checks that were run.


@kusari-inspector rerun - Trigger a re-analysis of this PR
@kusari-inspector feedback [your message] - Send feedback to our AI and team
See Kusari's documentation for setup and configuration.
Commit: 89a67a5, performed at: 2025-09-27T19:11:14Z

Found this helpful? Give it a 👍 or 👎 reaction!

@kusari-inspector

Copy link
Copy Markdown

Kusari PR Analysis rerun based on - 5af7c10 performed at: 2025-08-29T13:39:10Z - link to updated analysis

@chaseisabelle

Copy link
Copy Markdown
Contributor

Wow. Good catch 👍

@manuelarte

Copy link
Copy Markdown
Author

Any comments?

@kusari-inspector

Copy link
Copy Markdown

Kusari PR Analysis rerun based on - 970c583 performed at: 2025-09-13T19:18:26Z - link to updated analysis

@kusari-inspector

Copy link
Copy Markdown

Kusari PR Analysis rerun based on - 2ffe9cb performed at: 2025-09-24T19:14:50Z - link to updated analysis

@kusari-inspector

Copy link
Copy Markdown

Kusari PR Analysis rerun based on - 89a67a5 performed at: 2025-09-27T19:11:34Z - link to updated analysis

@manuelarte

Copy link
Copy Markdown
Author

Hi,

I wanted to follow up on this PR which has been open for about a month. I've noticed there's been recent activity in the project with other PRs being merged, so I'm wondering what the next steps might be for this one.

I am happy to make any necessary changes or provide additional context if that would help. If this isn't a priority right now or doesn't fit with the project's direction, I'd appreciate knowing that too—I'd rather have clear feedback than leave it open indefinitely.

Thanks!

@chaseisabelle

Copy link
Copy Markdown
Contributor

Hi,

I wanted to follow up on this PR which has been open for about a month. I've noticed there's been recent activity in the project with other PRs being merged, so I'm wondering what the next steps might be for this one.

I am happy to make any necessary changes or provide additional context if that would help. If this isn't a priority right now or doesn't fit with the project's direction, I'd appreciate knowing that too—I'd rather have clear feedback than leave it open indefinitely.

Thanks!

This project is maintained by very busy folks who are probably going to want to pour through the changes manually, especially with the rise of AI attackings, so it's going to take some time. I'm not sure of any way to expedite the process; however, if you can think of one, please let me/us know :)

Personal plug: #2070 wink+nudge

@manuelarte

Copy link
Copy Markdown
Author

Hi,
I wanted to follow up on this PR which has been open for about a month. I've noticed there's been recent activity in the project with other PRs being merged, so I'm wondering what the next steps might be for this one.
I am happy to make any necessary changes or provide additional context if that would help. If this isn't a priority right now or doesn't fit with the project's direction, I'd appreciate knowing that too—I'd rather have clear feedback than leave it open indefinitely.
Thanks!

This project is maintained by very busy folks who are probably going to want to pour through the changes manually, especially with the rise of AI attackings, so it's going to take some time. I'm not sure of any way to expedite the process; however, if you can think of one, please let me/us know :)

Personal plug: #2070 wink+nudge

Hi @chaseisabelle!

Thanks for your answer. I completely understand that maintainers are busy.

My main goal is just to clarify the status. At this point, I'd just like to know if:

  • This PR is still in the queue for a future review, or
  • It has been reviewed/checked and decided against merging.

If it's the latter, I'm totally fine with that—I'd just prefer to know so I can close the PR and don't need to invest time in keeping the branch updated.

Thanks again for your time!

@jamietanna

jamietanna commented Sep 28, 2025

Copy link
Copy Markdown
Member

Thanks for the contribution!

To clarify:

My main goal is just to clarify the status. At this point, I'd just like to know if:

  • This PR is still in the queue for a future review, or
  • It has been reviewed/checked and decided against merging.

It's the former.

If it were the latter, I'd have dropped a comment / closed the PR accordingly

Please be patient - we've got quite a few other higher-priority PRs, and will get to this at some point!

I'd just prefer to know so I can close the PR and don't need to invest time in keeping the branch updated.

No worries about keeping it updated, I'll get it updated when I get round to it :)

Related: #1909 (comment)

@manuelarte manuelarte closed this by deleting the head repository May 2, 2026
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