Comments (5)
Fixed in 55be821
from toml-test.
Your PR #138 fixed this (there's already a exp-leading-us
test, so no need to do anything else).
from toml-test.
PR #138 changed exp-trailing-us.toml
, which tests:
exp-trailing-us = 1e23_
trailing-us-exp.toml
tests:
trailing-us-exp = 1_e2
trailing-us-exp2 = 1.2_e2
Each test has an invalid underscore in different positions of the float. Assuming we want to test against all these scenarios, it seems to me that the tests in trailing-us-exp.toml
should be split out.
Normally I'd just submit a PR, but I think that adding/removing tests may require some changes in the binary in addition to just changing the tests, and I don't know the first thing about Go.
from toml-test.
adding/removing tests may require some changes in the binary in addition to just changing the tests, and I don't know the first thing about Go.
The entire directory is included; you don't need to do anything to add or remove a test other than create a file.
from toml-test.
Thank you for the fix!
The entire directory is included; you don't need to do anything to add or remove a test other than create a file.
I'll keep that in mind if I have future PRs.
Do the tests\files-toml-x
files need to be updated?
Ah, it looks like it's all auto-generated. Thanks again!
from toml-test.
Related Issues (20)
- Installation from conda-forge
- Bug in Test Case: valid/string/multiline-quotes HOT 2
- Add tests for unicode in bare keys
- Add test for tab character HOT 2
- Add tests for invalid encodings in all string types HOT 2
- Generate tests from the spec? HOT 8
- Add tests for "Permit more control characters in comments"
- Contradicting tests `invalid/no-seconds.toml` and `valid/no-seconds.toml` HOT 6
- Millisecond truncation is incorrect HOT 1
- Millisecond decoder tests pass with second precision output HOT 1
- Transfer to the toml-lang organisation? HOT 7
- valid/table/array-one fails when using inline array and table HOT 2
- Consider adding some performance tests HOT 1
- Invalid Unicode Escapes HOT 4
- [Valid Tests] Add Extra Spaces in Inline Tables HOT 1
- Clarify if - is permitted as a starting character in bare keys (toml 1.0.0) HOT 1
- Add tests for newlines within an array nested in an inline table
- Inconsistent test result of '+inf' HOT 4
- Add a test for oversized floating-point literals HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from toml-test.