More semconv features and cleanups

[jsuereth]

• Add new helper filter to split group ids into constituent parts, e.g. "registry.foo" => ["registry", "foo"]
• Update weaver_forge tests to work on windows
  • Translate"\r\n" => "\n" for compared output
  • Add filename filter to jinja templates that converts "" to "/"
  • Improve error output when file system comparison fails.
  • Add 'trim' to a few templates to help remove platform specific EOLs
• Update Dockerfile to use debug mode, so we get debug output on template failure

Fixes #84

[jsuereth]

Note: Overall weaver on windows tests still fail in other crates, mostly anything that prints file paths with expected output files.

[codecov]

Codecov Report

Attention: Patch coverage is 70.00000% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 63.6%. Comparing base (aa5a22f) to head (32ea520).

Files	Patch %	Lines
crates/weaver_forge/src/lib.rs	70.0%	3 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff          @@
##            main     #97   +/-   ##
=====================================
  Coverage   63.6%   63.6%           
=====================================
  Files         54      54           
  Lines       3235    3245   +10     
=====================================
+ Hits        2058    2065    +7     
- Misses      1177    1180    +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[lquerel]

Just 2 minor suggestions before approving this PR. @

[lquerel]

Is there a reason why split_id does not directly return a Vec to avoid the list filter?

[jsuereth]

My cluelessness in using Minijinja. If I can just return a Vec and it works, great. I thought that Value::from_seq + returning a Vec were basically the same with one being less obvious.

[jsuereth]

Moved to vec. Wierdly minijinja still is confused on values.

[lquerel]

Based on the code of the list filter (see below) you need to wrap the vec into a value.

#[cfg_attr(docsrs, doc(cfg(feature = "builtins")))]
pub fn list(state: &State, value: Value) -> Result<Value, Error> {
    let iter = ok!(state.undefined_behavior().try_iter(value).map_err(|err| {
        Error::new(ErrorKind::InvalidOperation, "cannot convert value to list").with_source(err)
    }));
    Ok(Value::from(iter.collect::<Vec<_>>()))
}

[lquerel]

Note: Overall weaver on windows tests still fail in other crates, mostly anything that prints file paths with expected output files.

I don't have access to Windows, unfortunately. Can you create a GH issue to describe what's causing problems on Windows? It could be a perfect issue for someone who wants to start contributing to this repo.

[jsuereth]

Note: Overall weaver on windows tests still fail in other crates, mostly anything that prints file paths with expected output files.

I don't have access to Windows, unfortunately. Can you create a GH issue to describe what's causing problems on Windows? It could be a perfect issue for someone who wants to start contributing to this repo.

I had one open for forge. I haven't investigate the issue for other crates you, but I'll open one when I'm back on my windows machine later.