Page MenuHomePhabricator

[Review ONLY] Migrate bitcoinabc.org codebase into the ABC repo
AbandonedPublic

Authored by jasonbcox on Jan 31 2020, 19:48.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Summary

This diff is for review purposes only. In order to preserve history, I will need to land
these commits via git push rather than arc land.

Note the files will be located at /web at the top level of the project.

Pulling in these commits will allow us to get better review on bitcoinabc.org improvements as well as build on our existing automation. Some examples of improvements coming down the pipe:

  • Reducing bitcoinabc.org deployment times from 10-15 minutes to ~5 seconds.
  • Reduced reliance on current hosting infra which, for reasons I won't go into here, is not viable for us in a few months.
  • Start linting web files (where appropriate).
  • Release automation can post release notes to the website. This is an item that has been requested by multiple parties over the years.

Note that George is working on improving the blog, which may include some of the reworkings that @deadalnix mentioned below. We need these improvements, but lack of dev resources makes it infeasible to tackle this all in one go prior to this migration.

Test Plan

Follow build instructions in README

Diff Detail

Repository
rABC Bitcoin ABC
Branch
abcorg-s-mv-rebase
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningbitcoinabc.org/_posts/2018-01-14-CashAddr-es.md:46SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-04-01-upgrade-es.md:23SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-05-07-incident-report-es.md:25SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-05-07-incident-report-es.md:27SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-08-15-benefits-of-ctor-es.md:10SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-08-15-benefits-of-ctor-es.md:16SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-08-20-announcing-bitcoin-abc-0-18-0-es.md:13SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-09-06-sharding-bitcoin-cash-es.md:19SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-09-06-sharding-bitcoin-cash-es.md:21SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-09-07-bitcoin-abc-and-the-block-size-limit-es.md:47SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-09-26-incident-report-es.md:10SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-09-26-incident-report-es.md:34SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/_posts/2018-09-26-incident-report-es.md:38SPELL1Possible Spelling Mistake
Warningbitcoinabc.org/css/bootstrap.css.map:1SPELL1Possible Spelling Mistake
Unit
No Test Coverage
Build Status
Buildable 9220
Build 16385: Default Diff Build & Tests
Build 16384: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Feb 1 2020, 02:31
deadalnix added a subscriber: deadalnix.

1/ Name it web; www or something.
2/ This needs to reworked. I don't think we want blogs posts in there for instance.

This revision now requires changes to proceed.Feb 1 2020, 02:31

Moved bitcoinabc.org -> web at the top level

2/ This needs to reworked. I don't think we want blogs posts in there for instance.

While this may be true, we don't have a webdev resource to work on this at the moment.
I'm migrating what we have right now so that I can continue working on our deployment
infrastructure and automate the version bumping that we do regularly on the website.

If we want to get rid of the blog-style posts, there are multiple things to consider
that I would rather not shift my focus to at this time. Just off the top of my head:

  • Providing a post archive so the hardlinks still function.
  • Do we move past and future blog posts a designated site or something else?
  • Without the posts/announcements section, the layout of the site is liable to change.

Hmm when you guys described moving the website into the repo I thought it was crazy as I was imagining something much bigger... this seems not so bad!

jasonbcox edited the summary of this revision. (Show Details)
jasonbcox edited the summary of this revision. (Show Details)
majcosta added inline comments.
web/.gitignore
10 ↗(On Diff #19413)

should the contents of the page's .gitignore file be moved to the main repo .gitignore?

web/.gitignore
10 ↗(On Diff #19413)

No. Nested .gitignores are not only supported, but also good design. Moving these up to the top-level would put a lot of garbage that does not belong in the top-level.

After a discussion in our team meeting, we decided it's not the right way forward. We'll add the bitcoinabc.org repo to Phab to improve code review and iteration throughput, but the repo itself will remain separate.