TOML parser error when date is followed by comment

[veddox]

If a TOML file includes a date, a trailing comment causes the parser to break. This was previously described here: https://discourse.julialang.org/t/toml-parsefile-error-on-reading-date-formatted-variable/59681

Minimum Working Example:

# mwe.toml
	
[core]
# these lines are fine
name = "mwe.toml" # here, comments are no problem
version = 1

[extension]
# the next line is fine as long as it has no trailing comment
startdate = 2020-01-01
enddate = 2020-01-05 # <<< this line causes an error
mode = "generic"

julia> using TOML

julia> TOML.parsefile("mwe.toml")
ERROR: TOML Parser error:
/home/xo30xoqa/Documents/Promotion/Persephone/src/mwe.toml:11:22 error: expected newline after key value pair
  enddate = 2020-01-05 # this line causes an error
                       ^                           
Stacktrace:
 [1] parse
   @ ./toml_parser.jl:441 [inlined]
 [2] parsefile(f::String)
   @ TOML /usr/local/lib/julia/share/julia/stdlib/v1.8/TOML/src/TOML.jl:43
 [3] top-level scope
   @ REPL[11]:1

Version info:

Julia Version 1.8.2
Commit 36034abf260 (2022-09-29 15:21 UTC)
Platform Info:
  OS: Linux (x86_64-linux-gnu)
  CPU: 8 × Intel(R) Core(TM) i5-10210U CPU @ 1.60GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-13.0.1 (ORCJIT, skylake)
  Threads: 1 on 8 virtual cores

(installed via download from official site)

[veddox]

I think this was actually fixed by #47448, specifically in toml_parser.jl lines 474 and 482, by including peek(l) == '#'. Unfortunately, I haven't yet managed to get the git version of Julia running yet to verify that the bug is gone.

[KristofferC]

The 1.9 alpha should have that included and can be downloaded from the homepage.

[veddox]

Can confirm, 1.9 fixes the error.