.busted

@@ -1,8 +1,5 @@
 return {
 	_all = {
 		lpath = "?.lua;?/init.lua"
-	};
+	}
-	core = {
-		recursive = false
-	};
 }

.editorconfig

@@ -1,2 +0,0 @@
-[*]
-indent_style = tab

.luarc.json

@@ -1,3 +0,0 @@
-{
-    "workspace.checkThirdParty": false
-}

glass-dev-1.rockspec

@@ -0,0 +1,30 @@
+package = "glass"
+version = "dev-1"
+source = {
+   url = "git+https://github.com/darkwiiplayer/glass"
+}
+description = {
+   homepage = "https://github.com/darkwiiplayer/glass",
+   license = "Unlicense"
+}
+dependencies = {
+	"luafilesystem"
+}
+build = {
+   type = "builtin",
+   modules = {
+      glass = "glass.lua",
+      ["glass.cosmo"] = "glass/cosmo.lua",
+      ["glass.csv"] = "glass/csv.lua",
+      ["glass.discount"] = "glass/discount.lua",
+      ["glass.environment"] = "glass/environment.lua",
+      ["glass.error"] = "glass/error.lua",
+      ["glass.json"] = "glass/json.lua",
+      ["glass.lua"] = "glass/lua.lua",
+      ["glass.raw"] = "glass/raw.lua",
+      ["glass.skooma.html"] = "glass/skooma/html.lua",
+      ["glass.skooma.xml"] = "glass/skooma/xml.lua",
+      ["glass.table"] = "glass/table.lua",
+      ["glass.yaml"] = "glass/yaml.lua",
+   }
+}

glass-dev-2.rockspec

@@ -1,32 +0,0 @@
-package = "glass"
-version = "dev-2"
-source = {
-	url = "git+https://github.com/darkwiiplayer/glass"
-}
-description = {
-	summary = "A library to load configuration files by accessing tables",
-	homepage = "https://github.com/darkwiiplayer/glass",
-	license = "Unlicense"
-}
-dependencies = {
-	"luafilesystem"
-}
-build = {
-	type = "builtin",
-	modules = {
-		glass = "glass.lua",
-		["glass.cosmo"] = "glass/cosmo.lua",
-		["glass.csv"] = "glass/csv.lua",
-		["glass.discount"] = "glass/discount.lua",
-		["glass.environment"] = "glass/environment.lua",
-		["glass.error"] = "glass/error.lua",
-		["glass.fennel"] = "glass/fennel.lua",
-		["glass.json"] = "glass/json.lua",
-		["glass.lua"] = "glass/lua.lua",
-		["glass.raw"] = "glass/raw.lua",
-		["glass.skooma.html"] = "glass/skooma/html.lua",
-		["glass.skooma.xml"] = "glass/skooma/xml.lua",
-		["glass.table"] = "glass/table.lua",
-		["glass.yaml"] = "glass/yaml.lua",
-	}
-}

glass.lua

@@ -21,8 +21,8 @@ function __metatable:__index(index)
 		return config.bind(path, self.__loaders)
 	else
 		for _, loader in ipairs(self.__loaders) do
-			local success, result = loader(path)
+			local result = loader(path)
-			if success then
+			if result then
 				rawset(self, index, result)
 				return result
 			end

glass/cosmo.lua

@@ -12,6 +12,8 @@ return function(name)
 	name = tostring(name) .. '.cosmo'
 	local text = raw(name)
 	if text then
-		return true, assert(cosmo.compile(text, name))
+		return assert(cosmo.compile(text, name))
+	else
+		return nil
 	end
 end

glass/csv.lua

@@ -10,6 +10,6 @@ local csv = require 'streamcsv'
 return function(name)
 	local file = io.open(name..'.csv')
 	if file then
-		return true, csv.file(file)
+		return csv.file(file)
 	end
 end

glass/discount.lua

@@ -14,8 +14,10 @@ return function(name)
 	local file = io.open(name)
 	if file then
 		local html = discount(file:read("*a"))
-		return true, function()
+		return function()
 			return html
 		end
+	else
+		return nil
 	end
 end

glass/environment.lua

@@ -4,5 +4,5 @@
 --- Loads values from environment variables
 -- @treturn string The value of the environment variable
 return function(name)
-	return true, os.getenv(name:match("[^/].+$"))
+	return os.getenv(name:match("[^/].+$"))
 end

glass/fennel.lua

@@ -1,18 +0,0 @@
---- Loader for Fennel files
--- @module glass.fennel
-
-local fennel = require "fennel"
-local read = require 'glass.raw'
-
---- Loads and compiles a Fennel file and runs it.
--- Returns the result of running the Fennel file.
--- If running the code immediately is not desired,
--- it has to be returned as a function.
--- @return The result of the Fennel file after running it
--- @function load
-return function(file)
-	local success, raw = read(file..'.fnl')
-	if success then
-		return true, fennel.eval(raw)
-	end
-end

glass/json.lua

@@ -11,8 +11,8 @@ local read = require 'glass.raw'
 -- @treturn table JSON-Data
 -- @function load
 return function(file)
-	local success, raw = read(file..'.json')
+	local raw = read(file..'.json')
-	if success then
+	if raw then
-		return true, json.decode(raw)
+		return json.decode(raw)
 	end
 end

glass/lua.lua

@@ -9,7 +9,5 @@
 -- @function load
 return function(name)
 	local f = loadfile(name..'.lua')
-	if f then
+	return f and f() or nil
-		return true, f()
-	end
 end

glass/moonhtml.lua

@@ -1,6 +1,7 @@
 --- Loader for MoonHTML files
 -- @module glass.moonhtml
 
+
 local template = require 'restia.template'
 
 --- Loads and compiles a moonhtml template.
@@ -10,7 +11,7 @@ return function(name)
 	name = tostring(name) .. '.moonhtml'
 	local file = io.open(name)
 	if file then
-		return true, assert(template.loadmoon(file:read("*a"), name))
+		return assert(template.loadmoon(file:read("*a"), name))
 	else
 		return nil
 	end

glass/raw.lua

@@ -6,10 +6,8 @@
 -- @function load
 return function(path)
 	local f = io.open(path)
-	if not f then
+	if not f then return end
-		return false
-	end
 	local result = f:read("*a")
 	f:close()
-	return true, result
+	return result
 end

glass/skooma/html.lua

@@ -17,12 +17,9 @@ end})
 -- @function load
 return function(name)
 	name = tostring(name)..'.html.skooma'
-	local template, err = loadfile(name, "t", env)
+	local template = loadfile(name, "t", env)
-	if template == nil then
-		return nil, err
-	end
 	if setfenv then
 		setfenv(template, env)
 	end
-	return true, template
+	return template
 end

glass/skooma/xml.lua

@@ -17,12 +17,9 @@ end})
 -- @function load
 return function(name)
 	name = tostring(name)..'.xml.skooma'
-	local template, err = loadfile(name, "t", env)
+	local template = loadfile(name, "t", env)
-	if template == nil then
-		return nil, err
-	end
 	if setfenv then
 		setfenv(template, env)
 	end
-	return true, template
+	return template
 end

glass/table.lua

@@ -7,9 +7,6 @@
 -- @treturn function A loader function to be used with `glass.bind`
 return function(input)
 	return function(name)
-		local result = input[name:match("[^/].+$")]
+		return input[name:match("[^/].+$")]
-		if result then
-			return true, result
-		end
 	end
 end

glass/yaml.lua

@@ -11,11 +11,8 @@ local read = require 'glass.raw'
 -- @treturn table YAML-Data
 -- @function load
 return function(file)
-	local success, raw = read(file..'.yml')
+	local raw = read(file..'.yml') or read(file..'.yaml')
-	if not success then
+	if raw then
-		success, raw = read(file..'.yaml')
+		return yaml.load(raw)
-	end
-	if success then
-		return true, yaml.load(raw)
 	end
 end

readme.md

@@ -31,7 +31,6 @@ Glass offers the following loaders out of the box:
 * `csv` loads csv files (with header line)
 * `discount` loads markdown files\*.
 * `environment` loads environment variables.
-* `fennel` loads and executes fennel files.
 * `json` loads a JSON file as a Lua table. (requires `cjson`)
 * `lua` loads and executes a Lua file.
 * `moonhtml` loads a MoonHTML template and returns it as a function.
@@ -54,8 +53,6 @@ file and attempts to load it into a Lua value.
 Loaders will typically add an extension to the given file name before checking
 whether that file exists and can be loaded.
 
-A successful loader should return `true` followed by its result.
+When a loader cannot find the expected file, it should return `nil` to let glass
-When a loader can't load a key, it should return `false` and an optional
+continue the loader chain. When a loader returns a truthy value, this will be
-description of why.
+used and no further loaders will be tried.
-When a loader encounters an error (file found but can't be parsed), it should
-error.

spec/fennel/fennel_loader_spec.moon

@@ -1,6 +0,0 @@
-describe 'fennel loader', ->
-	before_each -> export loader = require 'glass.fennel'
-	it 'loads Fennel files', ->
-		success, result = loader 'spec/fixtures/test'
-		assert.true success
-		assert.same { foo: 'bar', tab: {} }, result

spec/fixtures/test.fnl

@@ -1,2 +0,0 @@
-{"foo" "bar"
- "tab" []}

spec/loaders_spec.moon

@@ -1,11 +1,9 @@
 describe 'raw loader', ->
 	before_each -> export loader = require 'glass.raw'
 	it 'loads files as plain text', ->
-		assert.same {true, 'plain text\n'}, {loader 'spec/fixtures/test'}
+		assert.same 'plain text\n', loader 'spec/fixtures/test'
 
 describe 'lua loader', ->
 	before_each -> export loader = require 'glass.lua'
 	it 'loads Lua files', ->
-		success, result = loader 'spec/fixtures/test'
+		assert.same { foo: 'bar', tab: {} }, loader 'spec/fixtures/test'
-		assert.true success
-		assert.same { foo: 'bar', tab: {} }, result