Skip to content

Commit

Permalink
[WiP] rename project from Caravel to Superset (apache#1576)
Browse files Browse the repository at this point in the history
* Change in files

* Renamin files and folders

* cleaning up a single piece of lint

* Removing boat picture from docs

* add superset word mark

* Update rename note in docs

* Fixing images

* Pinning datatables

* Fixing issues with mapbox-gl

* Forgot to rename one file

* Linting

* v0.13.0

* adding pyyaml to dev-reqs
  • Loading branch information
mistercrunch authored Nov 10, 2016
1 parent 973537f commit 15b67b2
Show file tree
Hide file tree
Showing 408 changed files with 2,797 additions and 2,789 deletions.
16 changes: 8 additions & 8 deletions .codeclimate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ engines:
eslint:
enabled: true
config:
config: caravel/assets/.eslintrc
config: superset/assets/.eslintrc
pep8:
enabled: true
fixme:
Expand All @@ -19,17 +19,17 @@ engines:
ratings:
paths:
- "**.py"
- "caravel/assets/**.js"
- "caravel/assets/**.jsx"
- "superset/assets/**.js"
- "superset/assets/**.jsx"
exclude_paths:
- ".*"
- "**.pyc"
- "**.gz"
- "env/"
- "tests/"
- "caravel/assets/images/"
- "caravel/assets/vendor/"
- "caravel/assets/node_modules/"
- "caravel/assets/javascripts/dist/"
- "caravel/migrations"
- "superset/assets/images/"
- "superset/assets/vendor/"
- "superset/assets/node_modules/"
- "superset/assets/javascripts/dist/"
- "superset/migrations"
- "docs/"
9 changes: 5 additions & 4 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,22 +1,22 @@
*.pyc
caravel/assets/coverage/*
superset/assets/coverage/*
changelog.sh
.DS_Store
.coverage
_build
_static
_images
caravel/bin/caravelc
superset/bin/supersetc
env_py3
.eggs
build
*.db
tmp
caravel_config.py
superset_config.py
local_config.py
env
dist
caravel.egg-info/
superset.egg-info/
app.db
*.bak
.idea
Expand All @@ -27,3 +27,4 @@ app.db
*.js.map
node_modules
npm-debug.log
yarn.lock
4 changes: 2 additions & 2 deletions .landscape.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@ pep8:
full: true
ignore-paths:
- docs
- caravel/migrations/env.py
- caravel/ascii_art.py
- superset/migrations/env.py
- superset/ascii_art.py
ignore-patterns:
- ^example/doc_.*\.py$
- (^|/)docs(/|$)
6 changes: 3 additions & 3 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,10 @@ env:
before_install:
- npm install -g npm@'>=3.9.5'
before_script:
- mysql -e 'drop database if exists caravel; create database caravel DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci' -u root
- mysql -e 'drop database if exists superset; create database superset DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci' -u root
- mysql -u root -e "CREATE USER 'mysqluser'@'localhost' IDENTIFIED BY 'mysqluserpassword';"
- mysql -u root -e "GRANT ALL ON caravel.* TO 'mysqluser'@'localhost';"
- psql -c 'create database caravel;' -U postgres
- mysql -u root -e "GRANT ALL ON superset.* TO 'mysqluser'@'localhost';"
- psql -c 'create database superset;' -U postgres
- psql -c "CREATE USER postgresuser WITH PASSWORD 'pguserpassword';" -U postgres
- export PATH=${PATH}:/tmp/hive/bin
install:
Expand Down
58 changes: 29 additions & 29 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@ Look through the GitHub issues for features. Anything tagged with

### Documentation

Caravel could always use better documentation,
whether as part of the official Caravel docs,
Superset could always use better documentation,
whether as part of the official Superset docs,
in docstrings, `docs/*.rst` or even on the web as blog posts or
articles.

Expand All @@ -49,16 +49,16 @@ If you are proposing a feature:

## Latest Documentation

Latest documentation and tutorial are available [here](http://airbnb.io/caravel)
Latest documentation and tutorial are available [here](http://airbnb.io/superset)

## Setting up a Python development environment

Check the [OS dependencies](http://airbnb.io/caravel/installation.html#os-dependencies) before follows these steps.
Check the [OS dependencies](http://airbnb.io/superset/installation.html#os-dependencies) before follows these steps.

# fork the repo on GitHub and then clone it
# alternatively you may want to clone the main repo but that won't work
# so well if you are planning on sending PRs
# git clone [email protected]:airbnb/caravel.git
# git clone [email protected]:airbnb/superset.git

# [optional] setup a virtual env and activate it
virtualenv env
Expand All @@ -68,24 +68,24 @@ Check the [OS dependencies](http://airbnb.io/caravel/installation.html#os-depend
python setup.py develop

# Create an admin user
fabmanager create-admin --app caravel
fabmanager create-admin --app superset

# Initialize the database
caravel db upgrade
superset db upgrade

# Create default roles and permissions
caravel init
superset init

# Load some data to play with
caravel load_examples
superset load_examples

# start a dev web server
caravel runserver -d
superset runserver -d


## Setting up the node / npm javascript environment

`caravel/assets` contains all npm-managed, front end assets.
`superset/assets` contains all npm-managed, front end assets.
Flask-Appbuilder itself comes bundled with jQuery and bootstrap.
While these may be phased out over time, these packages are currently not
managed with npm.
Expand All @@ -112,14 +112,14 @@ export PATH="$HOME/.npm-packages/bin:$PATH"

#### npm packages
To install third party libraries defined in `package.json`, run the
following within the `caravel/assets/` directory which will install them in a
following within the `superset/assets/` directory which will install them in a
new `node_modules/` folder within `assets/`.

```
npm install
```

To parse and generate bundled files for caravel, run either of the
To parse and generate bundled files for superset, run either of the
following commands. The `dev` flag will keep the npm script running and
re-run it upon any changes within the assets directory.

Expand All @@ -135,7 +135,7 @@ For every development session you will have to start a flask dev server
as well as an npm watcher

```
caravel runserver -d -p 8081
superset runserver -d -p 8081
npm run dev
```

Expand All @@ -147,7 +147,7 @@ Python tests can be run with:

We use [Mocha](https://mochajs.org/), [Chai](http://chaijs.com/) and [Enzyme](http://airbnb.io/enzyme/) to test Javascript. Tests can be run with:

cd /caravel/caravel/assets/javascripts
cd /superset/superset/assets/javascripts
npm i
npm run test

Expand All @@ -157,13 +157,13 @@ Lint the project with:

# for python changes
flake8 changes tests
flake8 changes caravel
flake8 changes superset

# for javascript
npm run lint

## Linting with codeclimate
Codeclimate is a service we use to measure code quality and test coverage. To get codeclimate's report on your branch, ideally before sending your PR, you can setup codeclimate against your Caravel fork. After you push to your fork, you should be able to get the report at http://codeclimate.com . Alternatively, if you prefer to work locally, you can install the codeclimate cli tool.
Codeclimate is a service we use to measure code quality and test coverage. To get codeclimate's report on your branch, ideally before sending your PR, you can setup codeclimate against your Superset fork. After you push to your fork, you should be able to get the report at http://codeclimate.com . Alternatively, if you prefer to work locally, you can install the codeclimate cli tool.

*Install the codeclimate cli tool*
```
Expand Down Expand Up @@ -193,12 +193,12 @@ Generate the documentation with:
cd docs && ./build.sh

## CSS Themes
As part of the npm build process, CSS for Caravel is compiled from `Less`, a dynamic stylesheet language.
As part of the npm build process, CSS for Superset is compiled from `Less`, a dynamic stylesheet language.

It's possible to customize or add your own theme to Caravel, either by overriding CSS rules or preferably
It's possible to customize or add your own theme to Superset, either by overriding CSS rules or preferably
by modifying the Less variables or files in `assets/stylesheets/less/`.

The `variables.less` and `bootswatch.less` files that ship with Caravel are derived from
The `variables.less` and `bootswatch.less` files that ship with Superset are derived from
[Bootswatch](https://bootswatch.com) and thus extend Bootstrap. Modify variables in these files directly, or
swap them out entirely with the equivalent files from other Bootswatch (themes)[https://github.com/thomaspark/bootswatch.git]

Expand All @@ -221,14 +221,14 @@ meets these guidelines:

## Translations

We use [Babel](http://babel.pocoo.org/en/latest/) to translate Caravel. The
We use [Babel](http://babel.pocoo.org/en/latest/) to translate Superset. The
key is to instrument the strings that need translation using
`from flask_babel import lazy_gettext as _`. Once this is imported in
a module, all you have to do is to `_("Wrap your strings")` using the
underscore `_` "function".

To enable changing language in your environment, you can simply add the
`LANGUAGES` parameter to your `caravel_config.py`. Having more than one
`LANGUAGES` parameter to your `superset_config.py`. Having more than one
options here will add a language selection dropdown on the right side of the
navigation bar.

Expand All @@ -241,23 +241,23 @@ navigation bar.
As per the [Flask AppBuilder documentation] about translation, to create a
new language dictionary, run the following command:

pybabel init -i ./babel/messages.pot -d caravel/translations -l es
pybabel init -i ./babel/messages.pot -d superset/translations -l es

Then it's a matter of running the statement below to gather all stings that
need translation

fabmanager babel-extract --target caravel/translations/
fabmanager babel-extract --target superset/translations/

You can then translate the strings gathered in files located under
`caravel/translation`, where there's one per language. For the translations
`superset/translation`, where there's one per language. For the translations
to take effect, they need to be compiled using this command:

fabmanager babel-compile --target caravel/translations/
fabmanager babel-compile --target superset/translations/


## Adding new datasources

1. Create Models and Views for the datasource, add them under caravel folder, like a new my_models.py
1. Create Models and Views for the datasource, add them under superset folder, like a new my_models.py
with models for cluster, datasources, columns and metrics and my_views.py with clustermodelview
and datasourcemodelview.

Expand All @@ -267,6 +267,6 @@ to take effect, they need to be compiled using this command:

For example:

`ADDITIONAL_MODULE_DS_MAP = {'caravel.my_models': ['MyDatasource', 'MyOtherDatasource']}`
`ADDITIONAL_MODULE_DS_MAP = {'superset.my_models': ['MyDatasource', 'MyOtherDatasource']}`

This means it'll register MyDatasource and MyOtherDatasource in caravel.my_models module in the source registry.
This means it'll register MyDatasource and MyOtherDatasource in superset.my_models module in the source registry.
2 changes: 1 addition & 1 deletion INTHEWILD.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Please use [pull requests](https://github.com/airbnb/caravel/pull/new/master)
Please use [pull requests](https://github.com/airbnb/superset/pull/new/master)
to add your organization and/or project to this document!

Organizations
Expand Down
6 changes: 3 additions & 3 deletions ISSUE_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
Make sure these boxes are checked before submitting your issue - thank you!

- [ ] I have checked the caravel logs for python stacktraces and included it here as text if any
- [ ] I have reproduced the issue with at least the latest released version of caravel
- [ ] I have checked the superset logs for python stacktraces and included it here as text if any
- [ ] I have reproduced the issue with at least the latest released version of superset
- [ ] I have checked the issue tracker for the same issue and I haven't found one similar


### Caravel version
### Superset version


### Expected results
Expand Down
16 changes: 8 additions & 8 deletions MANIFEST.in
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
recursive-include caravel/templates *
recursive-include caravel/static *
recursive-exclude caravel/static/assets/node_modules *
recursive-include caravel/static/assets/node_modules/font-awesome *
recursive-exclude caravel/static/docs *
recursive-exclude caravel/static/spec *
recursive-include superset/templates *
recursive-include superset/static *
recursive-exclude superset/static/assets/node_modules *
recursive-include superset/static/assets/node_modules/font-awesome *
recursive-exclude superset/static/docs *
recursive-exclude superset/static/spec *
recursive-exclude tests *
recursive-include caravel/data *
recursive-include caravel/migrations *
recursive-include superset/data *
recursive-include superset/migrations *
Loading

0 comments on commit 15b67b2

Please sign in to comment.