Skip to content

Commit

Permalink
Add OSS website test, clean up test names, remove website dependency …
Browse files Browse the repository at this point in the history
…on sass

Reviewed By: ericvicenti

Differential Revision: D4726864

fbshipit-source-id: 38c13a08d42d11533a02c14a9acd5a05bbce307f
  • Loading branch information
Martin Konicek authored and facebook-github-bot committed Mar 30, 2017
1 parent c31a656 commit e82c2fb
Show file tree
Hide file tree
Showing 15 changed files with 2 additions and 2,552 deletions.
2 changes: 1 addition & 1 deletion scripts/validate-android-test-env.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ if [ -z "$(buck --version)" ]; then
echo "Your Buck install is broken."

if [ -d "/opt/facebook" ]; then
SUGGESTED="ff27d5270ecaa92727cd5a19954e62298fa78f09"
SUGGESTED="b9b76a3a5a086eb440a26d1db9b0731875975099"
echo "FB laptops ship with a Buck config that is not compatible with open "
echo "source. FB Buck requires the environment to set a buck version, but "
echo "the open source version of Buck forbids that."
Expand Down
3 changes: 1 addition & 2 deletions website/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,10 @@
"jstransform": "11.0.3",
"memory-cache": "^0.1.6",
"mkdirp": "^0.5.1",
"node-sass-middleware": "^0.11.0",
"optimist": "0.6.0",
"react": "~0.13.0",
"react-docgen": "^2.9.0",
"react-page-middleware": "git://github.com/facebook/react-page-middleware.git",
"react-page-middleware": "0.4.1",
"remove-markdown": "^0.1.0",
"request": "^2.69.0",
"semver-compare": "^1.0.0"
Expand Down
9 changes: 0 additions & 9 deletions website/server/server.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ const http = require('http');
const optimist = require('optimist');
const path = require('path');
const reactMiddleware = require('react-page-middleware');
const sassMiddleware = require('node-sass-middleware');

const argv = optimist.argv;

Expand Down Expand Up @@ -43,14 +42,6 @@ const buildOptions = {
};

const app = connect()
.use(sassMiddleware({
/* Options */
src: path.join(PROJECT_ROOT,'styles'),
dest: path.join(FILE_SERVE_ROOT,'react-native','css'),
response: false,
outputStyle: 'extended',
prefix: '/react-native/css',
}))
.use(function(req, res, next) {
// convert all the md files on every request. This is not optimal
// but fast enough that we don't really need to care right now.
Expand Down
186 changes: 0 additions & 186 deletions website/styles/_blog.scss

This file was deleted.

109 changes: 0 additions & 109 deletions website/styles/_footer.scss

This file was deleted.

56 changes: 0 additions & 56 deletions website/styles/_help.scss

This file was deleted.

Loading

2 comments on commit e82c2fb

@mkonicek
Copy link
Contributor

Choose a reason for hiding this comment

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

The reason to remove sass is it uses native code and adds complexity to the website build.

@henrikra
Copy link

Choose a reason for hiding this comment

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

So where all styles are now? :D

Please sign in to comment.