Skip to content

decaffinate and format#315

Closed
jeff-hykin wants to merge 14 commits intoatom-community:masterfrom
jeff-hykin:decaffinate-core
Closed

decaffinate and format#315
jeff-hykin wants to merge 14 commits intoatom-community:masterfrom
jeff-hykin:decaffinate-core

Conversation

@jeff-hykin
Copy link
Copy Markdown

just wanted to make a PR to run the tests

@aminya
Copy link
Copy Markdown
Member

aminya commented Aug 29, 2021

Please use decaffeinate instead of coffeescript itself

@jeff-hykin
Copy link
Copy Markdown
Author

Well I was trying to first compile it the same way it was compiled for runtime so that correctness would be ensured. Then I was gradually going to get it closer to being human readable like decaffeinate does.

However it looks like even this first attempt failed to build :/ so I'll have to go back and try not formatting them at all this time.

@aminya
Copy link
Copy Markdown
Member

aminya commented Sep 10, 2021

That's because you used CoffeeScript 2, which has some breaking changes compared to v1.

@jeff-hykin
Copy link
Copy Markdown
Author

Oh, I didn't realize Atom used v1. Thanks, that would've taken me forever to figure out

subrepo:
  subdir:   "acpm"
  merged:   "bbedc64fc"
upstream:
  origin:   "https://github.com/jeff-hykin/acpm"
  branch:   "master"
  commit:   "bbedc64fc"
git-subrepo:
  version:  "0.4.3"
  origin:   "???"
  commit:   "???"
subrepo:
  subdir:   "acpm"
  merged:   "2eaa38f15"
upstream:
  origin:   "https://github.com/jeff-hykin/acpm"
  branch:   "master"
  commit:   "2eaa38f15"
git-subrepo:
  version:  "0.4.3"
  origin:   "???"
  commit:   "???"
subrepo:
  subdir:   "acpm"
  merged:   "eb2e10fd4"
upstream:
  origin:   "https://github.com/jeff-hykin/acpm"
  branch:   "master"
  commit:   "eb2e10fd4"
git-subrepo:
  version:  "0.4.3"
  origin:   "???"
  commit:   "???"
@aminya aminya closed this Jun 22, 2022
Copy link
Copy Markdown
Member

@aminya aminya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we already discussed that acpm is not the way we want to go forward with apm. Since the decaffeination part is automatic, we can recreate this incrementally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants