specification icon indicating copy to clipboard operation
specification copied to clipboard

Examples README samples are broken

Open JBBianchi opened this issue 1 year ago • 9 comments
trafficstars

What seems off: The include of example sample seems to break the readme layout: image

What you expected to be: The layout to be nice and clean.

JBBianchi avatar Feb 17 '24 10:02 JBBianchi

Hey @JBBianchi thanks for reporting it! Are you willing to fix it? :D

ricardozanini avatar Feb 19 '24 18:02 ricardozanini

Yes, it's just a line return that needs to be added.

JBBianchi avatar Feb 20 '24 09:02 JBBianchi

It's broken again

RamiKassouf avatar Mar 18 '24 08:03 RamiKassouf

It's broken again

It hasn't been regenerated so far, so it's "still" broken :)

JBBianchi avatar Mar 18 '24 08:03 JBBianchi

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar May 03 '24 00:05 github-actions[bot]

@JBBianchi is this fixed?

ricardozanini avatar May 08 '24 13:05 ricardozanini

@JBBianchi is this fixed?

When https://github.com/serverlessworkflow/specification/pull/836 is merged we'll know for sure, but it should be.

JBBianchi avatar May 08 '24 13:05 JBBianchi

It looks like the workflow to hydrate the readme is broken: https://github.com/serverlessworkflow/specification/actions/workflows/examples-readme-hydration.yml

Either because the BOT_PAT secret is missing or the bot doesn't have the rights on the repository.

JBBianchi avatar May 08 '24 16:05 JBBianchi

That's weird, I'll take a look later

ricardozanini avatar May 08 '24 16:05 ricardozanini

@JBBianchi @ricardozanini Can you fix that for 0.9, meaning now, or should we close it as part of #843 ?

cdavernas avatar May 17 '24 08:05 cdavernas

I think it would be a good thing to fix the bot config. Even if the examples are dropped with the next version, we'll still need a functioning bot in the future.

JBBianchi avatar May 17 '24 08:05 JBBianchi

@ricardozanini Can you please address that?

cdavernas avatar May 17 '24 09:05 cdavernas

Closing this since we don't have this pipeline nor the examples README anymore.

ricardozanini avatar May 29 '24 17:05 ricardozanini

I think this must be fixed on 0.9.0

cdavernas avatar May 29 '24 21:05 cdavernas

@cdavernas I'll reopen but it might stale. I'd keep it close and if there's momentum we'll notice since the CI will fail, then we can fix it. I'm investing my time on 1.0.0 atm. Feel free to reopen.

ricardozanini avatar May 31 '24 12:05 ricardozanini

I'm investing my time on 1.0.0 atm

Not shit! 😆 Me too 🎉

cdavernas avatar May 31 '24 13:05 cdavernas