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.

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

Polkadot Runtime Releaser Milestone 1 #1236

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

aurexav
Copy link
Contributor

@aurexav aurexav commented Dec 28, 2024

Milestone Delivery Checklist

  • The milestone-delivery-template.md has been copied and updated.
  • This pull request is being made by the same account as the accepted application.
  • I have disclosed any and all sources of reused code in the submitted repositories and have done my due diligence to meet its license requirements.
  • In case of acceptance, invoices must be submitted and payments will be transferred to the Polkadot AssetHub and/or fiat account provided in the application.
  • The delivery is according to the Guidelines for Milestone Deliverables.

Link to the application pull request: w3f/Grants-Program#2373, w3f/Grants-Program#2474

@aurexav
Copy link
Contributor Author

aurexav commented Dec 28, 2024

It seems I reformatted the old deliverables, which caused the actions to fail?

Copy link
Member

@semuelle semuelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the delivery, @AurevoirXavier, but it looks like you are submitting changes to multiple unrelated files with your PR. It'd be great if your PR only included the delivery document you want to submit.

@PieWol
Copy link
Member

PieWol commented Jan 3, 2025

Hey @AurevoirXavier ,
I noticed that the default section for the link to your application document is not adjusted correctly. Please delete the default wording and link to your application file :)
Thanks!

@aurexav
Copy link
Contributor Author

aurexav commented Jan 3, 2025

Hey @AurevoirXavier , I noticed that the default section for the link to your application document is not adjusted correctly. Please delete the default wording and link to your application file :) Thanks!

I was still wondering why the CI failed. Stupid mistake. 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants