Skip to content

Commit

Permalink
Merge pull request #109 from dherman/toolchain-key
Browse files Browse the repository at this point in the history
Rename the `package.json` key to `"toolchain"`
  • Loading branch information
dherman authored Aug 5, 2018
2 parents 2b96435 + 6f59427 commit b5a625c
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion crates/notion-core/fixtures/basic/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
"@namespaced/something-else": "^6.3.7",
"eslint": "~4.8.0"
},
"notion": {
"toolchain": {
"node": "6.11.1",
"yarn": "1.2"
}
Expand Down
6 changes: 3 additions & 3 deletions crates/notion-core/src/serial/manifest.rs
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,11 @@ pub struct Manifest {
#[serde(rename = "devDependencies")]
pub dev_dependencies: HashMap<String, String>,

pub notion: Option<NotionManifest>,
pub toolchain: Option<ToolchainManifest>,
}

#[derive(Serialize, Deserialize)]
pub struct NotionManifest {
pub struct ToolchainManifest {
pub node: String,
pub yarn: Option<String>,
// FIXME: this should be in the notion config file
Expand All @@ -32,7 +32,7 @@ pub struct NotionManifest {

impl Manifest {
pub fn into_manifest(self) -> Fallible<Option<manifest::Manifest>> {
if let Some(notion) = self.notion {
if let Some(notion) = self.toolchain {
return Ok(Some(manifest::Manifest {
node: parse_requirements(&notion.node)?,
yarn: if let Some(yarn) = notion.yarn {
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
"devDependencies": {
"ember-cli": "2.18.1"
},
"notion": {
"toolchain": {
"node": "6.11.1",
"yarn": "1.7.0"
}
Expand Down

0 comments on commit b5a625c

Please sign in to comment.