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

'disabled' applied only when true, 'required' applied only when true and not disabled #84

Closed
wants to merge 2 commits into from

Conversation

ymarcon
Copy link
Contributor

@ymarcon ymarcon commented Nov 29, 2021

I am not sure for the required flag, as I was not able to test.
I had to upgrade esbuild because of evanw/esbuild#1711 (comment)

@mesqueeb
Copy link
Member

@ymarcon thanks so much. First look looks all good. Will merge and release later this week.
Is it ok only on next ?

@ymarcon
Copy link
Contributor Author

ymarcon commented Nov 30, 2021

Could you make a pre-release instead of (or in addition of) the @next snapshot ? What's your plan with releasing 1.0.0?

@mesqueeb
Copy link
Member

mesqueeb commented Dec 4, 2021

@ymarcon closed in favour of cherry pick:
https://github.com/CyCraft/blitzar/commits/v1.0.0
will release just as the latest version on next for now.

I plan to release 1.0.0 ideally in 2021 : D
Only couple things left to do:

  • finalise rewriting BlitzTable
  • clean up the repo
  • port over docs to VitePress

I'm working on BlitzTable during December.

@mesqueeb mesqueeb closed this Dec 4, 2021
@ymarcon ymarcon deleted the 83 branch December 6, 2021 08:46
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.

2 participants