diff options
Diffstat (limited to 'zarb-ml/mageia-dev/2012-September/018456.html')
| -rw-r--r-- | zarb-ml/mageia-dev/2012-September/018456.html | 74 |
1 files changed, 74 insertions, 0 deletions
diff --git a/zarb-ml/mageia-dev/2012-September/018456.html b/zarb-ml/mageia-dev/2012-September/018456.html new file mode 100644 index 000000000..081948133 --- /dev/null +++ b/zarb-ml/mageia-dev/2012-September/018456.html @@ -0,0 +1,74 @@ +<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN"> +<HTML> + <HEAD> + <TITLE> [Mageia-dev] [soft-commits] [5622] Code reorg. + </TITLE> + <LINK REL="Index" HREF="index.html" > + <LINK REL="made" HREF="mailto:mageia-dev%40mageia.org?Subject=Re%3A%20%5BMageia-dev%5D%20%5Bsoft-commits%5D%20%5B5622%5D%20Code%20reorg.&In-Reply-To=%3CCAONrEtZDSx976%3D3fD3VsEbsH8iojTQ4wBK4x00hmWyJqZpcRtw%40mail.gmail.com%3E"> + <META NAME="robots" CONTENT="index,nofollow"> + <META http-equiv="Content-Type" content="text/html; charset=us-ascii"> + <LINK REL="Previous" HREF="018458.html"> + <LINK REL="Next" HREF="018457.html"> + </HEAD> + <BODY BGCOLOR="#ffffff"> + <H1>[Mageia-dev] [soft-commits] [5622] Code reorg.</H1> + <B>Thierry Vignaud</B> + <A HREF="mailto:mageia-dev%40mageia.org?Subject=Re%3A%20%5BMageia-dev%5D%20%5Bsoft-commits%5D%20%5B5622%5D%20Code%20reorg.&In-Reply-To=%3CCAONrEtZDSx976%3D3fD3VsEbsH8iojTQ4wBK4x00hmWyJqZpcRtw%40mail.gmail.com%3E" + TITLE="[Mageia-dev] [soft-commits] [5622] Code reorg.">thierry.vignaud at gmail.com + </A><BR> + <I>Tue Sep 4 20:57:44 CEST 2012</I> + <P><UL> + <LI>Previous message: <A HREF="018458.html">[Mageia-dev] [soft-commits] [5663] debugging... +</A></li> + <LI>Next message: <A HREF="018457.html">[Mageia-dev] [soft-commits] [5660] new get_refined_packages_list(); fix get_submitted_packages() +</A></li> + <LI> <B>Messages sorted by:</B> + <a href="date.html#18456">[ date ]</a> + <a href="thread.html#18456">[ thread ]</a> + <a href="subject.html#18456">[ subject ]</a> + <a href="author.html#18456">[ author ]</a> + </LI> + </UL> + <HR> +<!--beginarticle--> +<PRE>On 4 September 2012 11:02, <<A HREF="https://www.mageia.org/mailman/listinfo/mageia-dev">root at mageia.org</A>> wrote: +><i> Code reorg. +</I>><i> +</I>><i> - CSS move in style.css +</I>><i> - configuration values move in conf.php +</I>><i> - function definitions move in lib.php +</I>><i> - a few code style fixes +</I>><i> - test_index.php to test these changes before pushing to prod file +</I> +It's a good practice to do smaller commits, aka separating code reorg +from actual style fixes. +Else nobody can actually review those style fixes as they're lost among +unrelated stuff (aka poor SNR) +</PRE> + + + + + + + +<!--endarticle--> + <HR> + <P><UL> + <!--threads--> + <LI>Previous message: <A HREF="018458.html">[Mageia-dev] [soft-commits] [5663] debugging... +</A></li> + <LI>Next message: <A HREF="018457.html">[Mageia-dev] [soft-commits] [5660] new get_refined_packages_list(); fix get_submitted_packages() +</A></li> + <LI> <B>Messages sorted by:</B> + <a href="date.html#18456">[ date ]</a> + <a href="thread.html#18456">[ thread ]</a> + <a href="subject.html#18456">[ subject ]</a> + <a href="author.html#18456">[ author ]</a> + </LI> + </UL> + +<hr> +<a href="https://www.mageia.org/mailman/listinfo/mageia-dev">More information about the Mageia-dev +mailing list</a><br> +</body></html> |