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.

Sign up for GitHub

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

Jump to bottom

Test with inline_express! #160

Merged
merged 7 commits into from
Nov 22, 2021
Merged

Test with inline_express! #160

merged 7 commits into from
Nov 22, 2021

Conversation

termoshtt
Copy link
Contributor

Rewrite ruststep/tests using inline_express! introduced in #158

@termoshtt termoshtt self-assigned this Nov 21, 2021
@termoshtt
Copy link
Contributor Author

termoshtt commented Nov 22, 2021 •
edited
Loading

There are two problems:

  1. Publicity is not coherent
  2. inline_express! for tests/any does not works correctly (this must be a bug related to Generate impl EntityTable<XxxAnyHolder> #135)

It will be hard to resolve them at once. I split 2. part into another PR (#161).

@termoshtt termoshtt mentioned this pull request Nov 22, 2021
Merged
@termoshtt termoshtt marked this pull request as ready for review November 22, 2021 08:23
@termoshtt termoshtt merged commit 184abf7 into master Nov 22, 2021
@termoshtt termoshtt deleted the test-with-inline-express branch November 22, 2021 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@termoshtt termoshtt

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@termoshtt