- 
                Notifications
    
You must be signed in to change notification settings  - Fork 2
 
2.1 #163
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
Conversation
in the process, also got rid of obsolete workarounds for legacy versions
replacing poor man's versions
shorthand does more harm than good
| 
           We could also add faucet-pipeline-css to the default plugins while we're at it 🤔  | 
    
          
 In a perfect world, I'd revisit the decision to have default plugins in the first place: As evidenced by this entire process, things seem overly entangled. Having said that, it might still be the right choice; I haven't really thought it through... So I'm a bit hesitant, but given where we stand now, your suggestion seems reasonable. Of course that also means we have to use a monorepo and, while we're at it, have we considered Kubernetes?  | 
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job, thank you!
| - 20.x | ||
| - 22.x | ||
| - 24.x | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| - 20.x | |
| - 22.x | |
| - 24.x | |
| - 20.x | |
| - 24.x | |
| - latest | 
Surely YAML can import such definitions from npm so we don't have to keep doing this everywhere? 😉
        
          
                CHANGELOG.md
              
                Outdated
          
        
      | notable changes for end users: | ||
| * added support for faucet-pipeline-assets | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| notable changes for end users: | |
| * added support for faucet-pipeline-assets | |
| notable changes for end users: | |
| * added support for faucet-pipeline-assets | 
for consistency 🙂
Oh, also, the date above is no longer correct...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was too optimistic 😄
| 
           Aside: GitHub only offers the "Rebase and merge" button, despite the history being linear for 2.x AFAICT - that seems like an unpleasant bug (because experience suggests it will actually rewrite history instead of fast-forwarding)?  | 
    
| 
           Version 2.1 is now released. The following faucet config will work now: export const assets = [{
		source: "./src/foo.txt",
		target: "./dist/foo.txt"
}];Now, I will need to update the documentation. I also noticed that the current version of faucet-pipeline-css does not work. So expect a PR on that as well 😂 Man, dusting off old projects sure releases a ton of dust. 
 I'm pretty sure we have used rebase and merge for ages. But I don't get what you are saying: The history is linear in this case (even though it is rewritten).  | 
    
This PR does not merge into the main branch, but into a branch based on the 2.0.0 release. My idea is to release a non-breaking release of faucet-pipeline-core. It contains:
The last one is the main reason for a 2.1.0 release. I released faucet-pipeline-assets today. It is the 2.2 branch of static with the following changes:
assetskey instead of the keywordstaticThe plugin already works in an ESM project:
Adding it to core removes the need for the first and last line.