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
Copy link
Copy Markdown

@manuelarte manuelarte commented Aug 29, 2025

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
Copy link
Copy Markdown

kusari-inspector Bot commented Aug 29, 2025

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
Copy link
Copy Markdown
Member

jamietanna commented Sep 28, 2025

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