Skip to content

Removed self-hosted YAML Serializer and PsychedYAML#3396

Closed
hsbt wants to merge 1 commit into
masterfrom
remove-self-hosted-yaml
Closed

Removed self-hosted YAML Serializer and PsychedYAML#3396
hsbt wants to merge 1 commit into
masterfrom
remove-self-hosted-yaml

Conversation

@hsbt

@hsbt hsbt commented Mar 17, 2020

Copy link
Copy Markdown
Member

Description:

Migrate from rubygems/bundler#6536

I hope to unify the specification of YAML serializer in Rubygems. But We have a some of the activated issues related to the default gems.


Tasks:

  • Describe the problem / feature
  • Write tests
  • Write code to solve the problem
  • Get code review from coworkers / friends

I will abide by the code of conduct.

@hsbt hsbt changed the title Removed psyched_yaml because the modern version of Ruby don't need it. Removed self-hosted YAML Serializer and PsychedYAML Mar 17, 2020
@simi simi force-pushed the remove-self-hosted-yaml branch from ff79d78 to 65f4ce1 Compare December 19, 2020 12:07
@simi

simi commented Dec 19, 2020

Copy link
Copy Markdown

@hsbt I have rebased this (also to restart CI since old logs were not available anymore). I hope that is fine.

Would it be possible to remove lib/bundler/psyched_yaml.rb as well?

@simi simi force-pushed the remove-self-hosted-yaml branch from 65f4ce1 to ef5c627 Compare December 19, 2020 12:31
@simi simi mentioned this pull request Dec 19, 2020
4 tasks
@hsbt

hsbt commented Nov 9, 2021

Copy link
Copy Markdown
Member Author

I gave up to migrate self-hosted YAML serializer to psych gem. Because we keep to specify any version with Gemfile. I have no idea to do it with psych as the default gems.

@hsbt hsbt closed this Nov 9, 2021
@hsbt hsbt deleted the remove-self-hosted-yaml branch November 9, 2021 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants